Skip to content

ci: add required-check migration helper - #3002

Closed
rwgk wants to merge 1 commit into
rwgk/maint/lychee-pr-nightlyfrom
rwgk/maint/pre-commit-ruleset-migration
Closed

rwgk wants to merge 1 commit into
rwgk/maint/lychee-pr-nightlyfrom
rwgk/maint/pre-commit-ruleset-migration

Conversation

@rwgk

@rwgk rwgk commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Description

Related to #2652. This draft contains an optional required-check migration
helper and its mocked tests. The standalone pre-commit workflow was introduced
in merged #2994; #2993 implements the independent documentation link checker.

Add a read-only-by-default CLI to preview replacing the pre-commit.ci - pr
required check with Pre-commit (Linux), Pre-commit (Windows), and
Documentation links from GitHub Actions. Preserve unrelated rules, required
checks, enforcement, branch conditions, and bypass actors. Reject inherited
organization rulesets and unexpected App integration IDs.

Writing requires --apply, workflows merged into main, and successful
representative checks from the expected App and owning workflows. Re-read
affected rulesets immediately before writing and reject concurrent changes.
Merging this PR does not run the helper or change required checks or App access.

The migration guide has been removed from the stack and preserved
at its historical revision.
The current administrative plan is a manual cutover. This helper edits existing
rulesets; it does not create a main-only replacement ruleset. The manual
cutover does not require this helper.

Validation

The helper and its mocked tests are unchanged by the documentation cleanup.
Original local validation through Pixi on PATH with Python 3.14 passed all 123
standalone CI-tool tests and 36 subtests, including the helper and link-input
tests. The mocked tests do not change GitHub settings.

Earlier hosted validation on 7e1a772a413f34e7031ca5382f6bd80940cdd39b passed
the pre-commit workflow
on Linux and Windows and the
documentation workflow
for authored links, rendered HTML, and the aggregate gate.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@rwgk rwgk added this to the cuda.core next milestone Oct 2, 2026
@rwgk rwgk added CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module labels Oct 2, 2026
@copy-pr-bot

copy-pr-bot Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rwgk rwgk self-assigned this Oct 2, 2026
@rwgk
rwgk added this pull request to stack #2995 October 2, 2026 17:40
@rwgk
rwgk force-pushed the rwgk/maint/pre-commit-ruleset-migration branch from 7e1a772 to 0fd6e02 Compare October 2, 2026 18:46
@rwgk
rwgk force-pushed the rwgk/maint/pre-commit-ruleset-migration branch 4 times, most recently from 708afa5 to 20880bf Compare October 3, 2026 18:47
@rwgk
rwgk force-pushed the rwgk/maint/pre-commit-ruleset-migration branch from 20880bf to 0c08636 Compare October 3, 2026 19:04
@rwgk rwgk changed the title ci: add required-check migration tooling and cutover guide ci: add required-check migration helper Oct 3, 2026
@rwgk

rwgk commented Oct 3, 2026

Copy link
Copy Markdown
Contributor Author

The helper here is one-time use only. Closing this PR, which will still keep the code around for reference by an admin, as an alternative to clicking buttons in the github web UI.

@rwgk rwgk closed this Oct 3, 2026
An error occurred while trying to automatically change base from rwgk/maint/lychee-pr-nightly to main October 13, 2026 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant