Skip to content

docs: explain pull request verification routing - #47

Merged
wolfiesch merged 3 commits into
mainfrom
ci/fast-lane-proof-20260831
Sep 1, 2026
Merged

docs: explain pull request verification routing#47
wolfiesch merged 3 commits into
mainfrom
ci/fast-lane-proof-20260831

Conversation

@wolfiesch

@wolfiesch wolfiesch commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • document the exact lightweight and full-lane path policy
  • keep verify as the stable branch-protection check
  • keep exact-head review enforcement in the repository shipping command
  • remove the review-event status bridge because bot-triggered runs completed as action_required without jobs

Verification

  • python3 .github/scripts/test_ci_policy.py
  • actionlint .github/workflows/ci.yml
  • commit b52fa907d989b94bdf991df06fdf3776eb2ff4d9 exercised the fast lane: classification passed in 10 seconds, verify passed in 3 seconds, and cross-platform jobs were skipped
  • the final head intentionally uses the full lane because it deletes a workflow file

Copilot AI lite review requested due to automatic review settings September 1, 2026 02:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new documentation has a couple of factual/precision issues (fast-lane path criteria and exact-head-review publication semantics) that should be corrected to match the implemented CI policy.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Documents how AgentTab’s CI policy routes documentation-only pull requests through a lightweight verification lane, and how an exact-head automated review check is enforced via branch protection.

Changes:

  • Add a “Pull request routing” section describing fast vs full verification lanes.
  • Describe the exact-head-review gate semantics tied to the pull request head commit.
File summaries
File Description
docs/verification.md Adds documentation for CI lane routing and the exact-head automated review check behavior.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/verification.md Outdated
Comment thread docs/verification.md Outdated
Copilot AI review requested due to automatic review settings September 1, 2026 02:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new routing doc references an unnamed “repository shipping command” enforcing exact-head review, which is not discoverable in-repo and should be clarified or linked.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread docs/verification.md Outdated
Copilot AI review requested due to automatic review settings September 1, 2026 02:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation matches the implemented CI lane selection and verify workflow behavior, and the removed workflow has no remaining in-repo references.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@wolfiesch
wolfiesch merged commit c4c437a into main Sep 1, 2026
9 checks passed
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.

2 participants