Skip to content

Document fork CI for contributors and maintainers - #74825

Open
potiuk wants to merge 2 commits into
fork-ci-4-auditfrom
fork-ci-5-docs
Open

potiuk wants to merge 2 commits into
fork-ci-4-auditfrom
fork-ci-5-docs

Conversation

@potiuk

@potiuk potiuk commented Oct 15, 2026

Copy link
Copy Markdown
Member

Stacked on #74823. Part 5 of 5 of the AIP-120 implementation (design: #73378); this PR's base is the previous branch, so the diff shows only this layer.

Adds "CI in your fork" to the contribution workflow (setup, breeze ci audit, what contributors see, joining and leaving the pilot), links it from the quick starts, PR guidelines and git conventions, adds the reconciler/triage ownership split to the maintainer triage doc, and the fork push run to 05_workflows.md.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 5)

Generated-by: Claude Code (Opus 5) following the guidelines

🤖 Generated with Claude Code

@potiuk
potiuk added this pull request to stack #74827 October 15, 2026 22:48
@potiuk
potiuk removed this pull request from stack #74827 October 15, 2026 23:26
@potiuk
potiuk changed the base branch from fork-ci-4-audit to main October 15, 2026 23:26
@potiuk
potiuk changed the base branch from main to fork-ci-4-audit October 15, 2026 23:30
@potiuk
potiuk added this pull request to stack #74833 October 15, 2026 23:30
@potiuk
potiuk force-pushed the fork-ci-5-docs branch 2 times, most recently from 0427b08 to 233d038 Compare October 16, 2026 22:18
Comment thread contributing-docs/03_contributors_quick_start.rst
Comment thread contributing-docs/05_pull_requests.rst Outdated
Comment thread contributing-docs/18_contribution_workflow.rst Outdated
Comment thread contributing-docs/18_contribution_workflow.rst Outdated
Comment thread contributing-docs/05_pull_requests.rst Outdated
Contributors need to know how to set up CI in their fork, how to check that setup, and how to join or leave the pilot, and maintainers need to know which parts of a gated pull request's lifecycle the reconciler owns so triage does not fight it. The previous one-line advice to enable GitHub Actions in the fork did not say any of that.

Generated-by: Claude Opus 5
While the gate is off, a contributor outside the pilot who enabled fork CI as the docs said got CI twice on every pull request. The docs now say fork CI applies to pilot participants and use fork ci pull requests until it is switched on for everyone, and the maintainer docs cover the needs project ci label, the CI passed status and why removing fork ci required is not enough.

Generated-by: Claude Opus 5

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants