Skip to content

Onboard new CI checks (alpha tier) - #549

Merged
sakanni merged 2 commits into
developfrom
CI_Toolkit_Proxy-#13-OnboardNewCIChecks
Jul 27, 2026
Merged

Onboard new CI checks (alpha tier)#549
sakanni merged 2 commits into
developfrom
CI_Toolkit_Proxy-#13-OnboardNewCIChecks

Conversation

@sakanni

@sakanni sakanni commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Adds the alpha-tier CI workflow as part of the new CI onboarding pilot for framework repositories.

The checks run on pull requests and are non-blocking during the evaluation period. On this PR they will pass within seconds since no code changes: the first code PR after merging exercises them fully.

Details and discussion: BuroHappoldEngineering/CI_Toolkit_Proxy#13

@sakanni
sakanni requested a review from a team July 14, 2026 14:41

@adecler adecler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Repo is part of the installer (to be checked) so it should be in the beta tier.

@peterjamesnugent

Copy link
Copy Markdown
Member

Test_Toolkit isn't in the installer:
https://github.com/BHoM/BHoM_Installer/blob/develop/IncludedRepos/include.txt

Agree it should probably have beta checks but I feel like in the past it has failed checks (e.g. compliance) because it is set up differently to traditional toolkits (and has a different function).

@sakanni sakanni added the type:compliance Non-conforming to code guidelines label Jul 15, 2026
@sakanni

sakanni commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Historical PRs + existing branch protection settings all characterise Test_Toolkit as an alpha marked repository i.e., only project, copyright, and serialisation checks run. New workflow only adds build (core) to the list of checks per the docs.

adecler
adecler previously approved these changes Jul 21, 2026

@adecler adecler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After talking with @sakanni , I'm happy with the Test toolkit staying in this tier.
We can always re-assess later to see if we want a broader availability of the Test toolkit later.

@cursor

cursor Bot commented Jul 24, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@sakanni

sakanni commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

@BHoMBot check required

@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

@sakanni to confirm, the following actions are now queued:

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check core
  • check null-handling
  • check serialisation
  • check versioning
  • check installer

There are 1 requests in the queue ahead of you.

@sakanni

sakanni commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

@BHoMBot check copyright-compliance

@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

@sakanni to confirm, the following actions are now queued:

  • check copyright-compliance

There are 7 requests in the queue ahead of you.

@sakanni
sakanni requested review from peterjamesnugent and removed request for peterjamesnugent July 24, 2026 12:49
@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

This repository is not part of the beta package and does not require a versioning check to be performed.

@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

This repository is not part of the installer package, and does not require an installer check.

@sakanni

sakanni commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

@BHoMBot check required

@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

@sakanni to confirm, the following actions are now queued:

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check core
  • check null-handling
  • check serialisation
  • check versioning
  • check installer

@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

This repository is not part of the beta package and does not require a versioning check to be performed.

@bhombot-ci

bhombot-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

This repository is not part of the installer package, and does not require an installer check.

@adecler adecler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All expected jobs added with the ci prefix

@sakanni
sakanni merged commit d77c1aa into develop Jul 27, 2026
13 checks passed
@sakanni
sakanni deleted the CI_Toolkit_Proxy-#13-OnboardNewCIChecks branch July 27, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:compliance Non-conforming to code guidelines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants