Skip to content

docs(scheduler): describe the pre-review refresh hold; correct the discarded-check figures - #1940

Draft
seonghobae wants to merge 1 commit into
mainfrom
docs/scheduler-pre-review-hold-followup
Draft

seonghobae wants to merge 1 commit into
mainfrom
docs/scheduler-pre-review-hold-followup

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Docs-only follow-up to #1937 (refs #1935). No workflow, script, or test changes.

What changes

Verification

Full gate on this head with GITHUB_ACTIONS=true: 2896 passed / 1 skipped, coverage report --fail-under=100 → 100%, interrogate → 100%; the 27 doc-contract tests (test_pr_governance_audit_contract.py and the rollout/changelog contracts) pass.

Not claimed

🤖 Generated with Claude Code

…scarded-check figures

docs/org-required-workflow-rollout.md described only the post-approval
update-branch posture; the pre-review refresh (now held while current-head
checks are in flight, #1937) is stated next to it. CHANGELOG's "22/28" and
"21/30" counted phantom check-runs created and cancelled in the same
instant; the evidence actually discarded per push was 10 and 11.

Refs #1935

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 50 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 988dc978-071d-49de-bba8-43d8c68c1ab8

📥 Commits

Reviewing files that changed from the base of the PR and between f2f91b8 and d624510.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • docs/org-required-workflow-rollout.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Contributor Author

2026-09-20 protected-base admission correction for d624510b6de79b8b101310d3431b0610adb376f9.

GitHub reports this Ready PR mechanically non-mergeable. Exact comparison with protected main is diverged and 361 commits behind; historical Checks are GREEN but no formal review exists. It is therefore not eligible for ordinary integration at this head/base.

Moving it to Draft / Proposed preserves every commit, valid delta, review, and thread. This is not closure or abandonment. Reconcile protected main through an ordinary non-force merge, repair valid findings, run terminal exact-head protection, and obtain qualifying independent current-head approval before returning to Ready. No review dismissal, synthetic status, manual rerun, bypass, Force Push, merge, or Close is authorized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: docs Documentation, ADR, PRD, or technical writing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant