Skip to content

fix: ignore reverted commit pairs - #2873

Open
iamhaseebn wants to merge 1 commit into
googleapis:mainfrom
iamhaseebn:fix-ignore-reverted-commits
Open

fix: ignore reverted commit pairs#2873
iamhaseebn wants to merge 1 commit into
googleapis:mainfrom
iamhaseebn:fix-ignore-reverted-commits

Conversation

@iamhaseebn

Copy link
Copy Markdown

Thank you for reviewing.

  • An issue was opened before implementation
  • Tests and linter pass
  • Code coverage does not decrease
  • No documentation update is necessary

Summary

  • Remove an unreleased commit and its matching revert from commit analysis
  • Support standard This reverts commit <sha> messages and explicit Revert: <sha> footers
  • Preserve reverts whose target is outside the current release range or whose abbreviated SHA is ambiguous
  • Handle revert-of-revert chains so the original change is restored

Testing

  • npm test - 1,216 passing
  • npm run fix - 0 errors, with 21 existing warnings in unrelated files
  • npm run compile

Fixes #296

@iamhaseebn
iamhaseebn requested review from a team as code owners August 9, 2026 09:27
@google-cla

google-cla Bot commented Aug 9, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@product-auto-label product-auto-label Bot added the size: l Pull request size is large. label Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ignore reverted commits

1 participant