Skip to content

Bug 2088987 - Bug shouldn't be closed when github commit and backout arrive on the same message - #2813

Merged
dklawren merged 4 commits into
mozilla:masterfrom
dklawren:2088987
Nov 7, 2026
Merged

dklawren merged 4 commits into
mozilla:masterfrom
dklawren:2088987

Conversation

@dklawren

@dklawren dklawren commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

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

No active review issues remain for the changes preventing revert and backout pushes from incorrectly closing bugs.

No open issues 🎉

What changed in this PR

Prevents GitHub push notifications from incorrectly closing a bug when the push includes a revert or backout for that bug, including pushes that mix fixes and reversions.

Changes:

  • Detect revert and backout commit titles and suppress automatic bug closure.
  • Explain in the automated comment when a revert or backout prevents closure.
  • Add regression tests for revert-only and mixed pushes, plus normal titles containing “revert.”
File Description
qa/​t/​rest_github_push_comment.t Adds helpers and regression tests for comment text and bug status.
Bugzilla/​API/​V1/​Github.pm Tracks revert/backout commits and uses them to guard automatic closure.

Review effort: Balanced

@dklawren
dklawren requested review from Xzzz and cgsheeh November 4, 2026 21:18
Comment thread Bugzilla/API/V1/Github.pm Resolved
Comment thread qa/t/rest_github_push_comment.t Resolved

@Xzzz Xzzz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Approved, with two small inline comments

@dklawren
dklawren merged commit d1e9dbd into mozilla:master Nov 7, 2026
7 checks passed
@dklawren
dklawren deleted the 2088987 branch November 7, 2026 01:43
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.

4 participants