Skip to content

test(pr-validation): gate fixture 1 of 3 — approve this one - #942

Closed
bedatty wants to merge 1 commit into
developfrom
test/gate-approve
Closed

bedatty wants to merge 1 commit into
developfrom
test/gate-approve

Conversation

@bedatty

@bedatty bedatty commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Description

Fixture 1 of 3 — approve this one. Opened to exercise the breaking-change gate on the post-#940 code. Not for merge: close it when the run is done.

The break is declared with a ! in the subject (feat(pr-validation)!:). Expected: the gate holds this at the breaking-approval Environment and asks for a decision.

What to watch after approving:

  • the comment rewrites to Breaking change approved, naming the approver as a link
  • breaking-change stays on the pull request — it is a fact, and approving does not make the break go away
  • awaiting-breaking-approval comes off — the decision is made
  • pushing another non-breaking commit does not ask again: the breaking set is unchanged, and after fix(pr-validation): close two paths past the breaking-change gate #940 that is confirmed by resolving the set from the approving run's own head rather than from the record

Type of Change

  • test: Adding or updating tests

Breaking Changes

Declared deliberately, to trip the gate. Nothing is released from a fixture branch.

Breaking change acknowledged: I understand that this PR intentionally introduces a breaking change and requires the next release to be a major version.

Testing

  • YAML syntax validated locally

This is the test.

Related Issues

None.

Declared with a bang in the subject. Fixture only; close, never merge.
@bedatty
bedatty requested a review from a team as a code owner October 20, 2026 13:46
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

The included review limit has been reached and this organization has disabled usage-based review continuation. Wait for reviews to reset or ask a billing admin to change After included review limits.

  • Ask an admin to enable usage-based reviews

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Next included review available in 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 92 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: LerianStudio/github-actions-shared-workflows/.coderabbit.yml
  • Review profile: ASSERTIVE
  • Plan: Essentials
  • Run ID: 818dfb9e-1987-4a7e-b12e-7e9f6d4ead71
📥 Commits

Reviewing files that changed from the base of the PR and between 6eeb4d8 and df7e6ea.

📒 Files selected for processing: 1
  • docs/.fixtures/gate-approve.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@lerian-studio lerian-studio added the size/XS PR changes < 50 lines label Oct 20, 2026
@lerian-studio

lerian-studio commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Note

✅ Breaking change approved

Approved by @ferr3ira-gabriel. 💥 The next release will be a major version.

Gated by rule develop <- '*' · How this gate works

@lerian-studio lerian-studio added documentation Improvements or additions to documentation breaking-change Callers must update their workflow configuration after this change labels Oct 20, 2026
@bedatty
bedatty deployed to breaking-approval October 20, 2026 13:47 — with GitHub Actions Active
@lerian-studio

Copy link
Copy Markdown
Contributor

🔍 Lint Analysis

Check Files Scanned Status
YAML Lint no changes ⏭️ skipped
Action Lint no changes ⏭️ skipped
Pinned Actions no changes ⏭️ skipped
Markdown Link Check 1 file(s) ✅ success
Spelling Check 1 file(s) ✅ success
Shell Check no changes ⏭️ skipped
README Check no changes ⏭️ skipped
Composite Schema no changes ⏭️ skipped
Deployment Matrix no changes ⏭️ skipped

🔍 View full scan logs

@lerian-studio

Copy link
Copy Markdown
Contributor

🔍 PR Validation Summary

✅ PR Mergeable — no blocking failures

Check Status Blocking
Source Branch ✅ success yes
PR Title ✅ success yes
PR Description ✅ success yes
Breaking Change Guard ✅ success yes
Commit Signatures ✅ success yes
PR Size ✅ success no
Auto Labels ✅ success no
PR Metadata ✅ success no

🔍 View workflow run

@lerian-studio

Copy link
Copy Markdown
Contributor

@coderabbitai review

@lerian-studio lerian-studio added the review-ready Required checks passed — CodeRabbit is cleared to review label Oct 20, 2026
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@bedatty bedatty closed this Oct 20, 2026

This branch was successfully deployed

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

Labels

breaking-change Callers must update their workflow configuration after this change documentation Improvements or additions to documentation review-ready Required checks passed — CodeRabbit is cleared to review size/XS PR changes < 50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants