Skip to content

test(pr-validation): gate fixture 2 of 3 — reject this one - #943

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

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

Conversation

@bedatty

@bedatty bedatty commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Description

Fixture 2 of 3 — reject this one. Opened to exercise the breaking-change gate on the post-#940 code. Not for merge: the gate should close it for us.

The break is declared in a footer rather than the subject, so this also exercises the second detection form.

What to watch after rejecting:

  • the comment rewrites to Breaking change rejected, naming the rejecter as a link, with the reason if one was given
  • the pull request is closed automatically (close_on_rejection), and the branch is left untouched
  • breaking-change stays, awaiting-breaking-approval comes off — same reasoning as approval: the break is still a fact

Type of Change

  • test: Adding or updating tests

Breaking Changes

Declared deliberately, in a footer, 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.

The break is declared in a footer rather than the subject, so this also
checks the second detection form.

BREAKING CHANGE: 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

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: LerianStudio/github-actions-shared-workflows/.coderabbit.yml
  • Review profile: ASSERTIVE
  • Plan: Essentials
  • Run ID: aef809dc-dc41-4c2c-954e-fc80a8c40dc3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • 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 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Caution

⛔ Breaking change rejected

Rejected by @ferr3ira-gabriel.

This pull request was closed. The branch is untouched — rework it without the break and reopen.

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

@lerian-studio lerian-studio added breaking-change Callers must update their workflow configuration after this change size/XS PR changes < 50 lines documentation Improvements or additions to documentation labels Oct 20, 2026
@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

🔍 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

This branch had an error being deployed

1 failed 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 size/XS PR changes < 50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants