Skip to content

ci: add required-check migration tooling and cutover guide - #3002

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

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. Depends on #2993's documentation link workflow and #2994's
standalone pre-commit workflow.

Add a read-only-by-default CLI to preview replacing the pre-commit.ci - pr
required check with the GitHub Actions checks Pre-commit (Linux),
Pre-commit (Windows), and Documentation links. 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, successful
representative checks bound to the expected App, and successful owning workflow
runs. Re-read affected rulesets immediately before writing and reject changes
made since discovery. Add mocked tests and preview/cutover/validation
instructions to the existing workflow guide.

The helper, tests, and cutover instructions were extracted unchanged from
#2993 so reviewers can assess repository-setting writes separately from the
workflow runtime. Merging this PR does not execute the helper or change live
required checks or App access. The existing service stays active; applying the
cutover requires separate authorization.

Validation through PATH Pixi with Python 3.14: all 123 standalone CI-tool tests
and 36 subtests pass, including the migration helper and link-input tests.
Repository pre-commit checks pass with the workflow's existing lychee and
local hook-installation reminder skips. The migration helper and its tests
are byte-identical to the pre-split implementation.

Hosted validation on 7e1a772a413f34e7031ca5382f6bd80940cdd39b: the
pre-commit workflow
passes on Linux and Windows, and the
documentation workflow
passes authored links, rendered HTML, and the aggregate gate.

This PR stays draft. Heavyweight CI remains under /ok to test <full-SHA>
control.

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 2 times, most recently from 55aed80 to 6014d3e Compare October 3, 2026 04:13
@rwgk
rwgk force-pushed the rwgk/maint/pre-commit-ruleset-migration branch from 6014d3e to 708afa5 Compare October 3, 2026 04:14

This branch has not been deployed

No deployments
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