Skip to content

ci: require the Node 22/24 and npm-consumer checks on main - #41

Merged
code-yeongyu merged 1 commit into
mainfrom
ci/ruleset-required-checks
Sep 24, 2026
Merged

code-yeongyu merged 1 commit into
mainfrom
ci/ruleset-required-checks

Conversation

@code-yeongyu

@code-yeongyu code-yeongyu commented Sep 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Sync .github/branch-ruleset.json with the live main protection ruleset. Its required status checks now match the job names from chore: refresh dependencies and move CI to Bun 1.4.2 #40: test (ubuntu-latest|macos-latest · node 22|24) and npm consumer (ubuntu-latest · node 22).
  • The live ruleset still required test (… · node 20) contexts, which no workflow has produced since the matrix moved to Node 22. Every PR stayed blocked, so chore: refresh dependencies and move CI to Bun 1.4.2 #40 had to be admin-merged. The live ruleset has already been updated through the rulesets API. This PR only updates the checked-in copy.

Verification

  • All five contexts reported success on merge commit fdaacf8 (run 35966155390).
  • No code changes.

Review in cubic

@code-yeongyu

Copy link
Copy Markdown
Owner Author

Merging with the admin bypass. All five required checks now pass under the updated ruleset (run 35967245194). The only remaining block is the required approving review, which the PR author cannot give.

@code-yeongyu
code-yeongyu merged commit 16d73b1 into main Sep 24, 2026
7 checks passed
@code-yeongyu
code-yeongyu deleted the ci/ruleset-required-checks branch September 24, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant