Skip to content

chore: remove duplicate suppression rules from base that are in the generated branch#8138

Merged
jeremylong merged 1 commit intomainfrom
scratch/cleanup-suppression-rules-2
Nov 20, 2025
Merged

chore: remove duplicate suppression rules from base that are in the generated branch#8138
jeremylong merged 1 commit intomainfrom
scratch/cleanup-suppression-rules-2

Conversation

@jeremylong
Copy link
Copy Markdown
Collaborator

Removes duplicate suppressions from base file that exist in the generated branch.

@boring-cyborg boring-cyborg bot added the core changes to core label Nov 18, 2025
@jeremylong jeremylong added this to the 12.2.0 milestone Nov 18, 2025
Copy link
Copy Markdown
Collaborator

@chadlwilson chadlwilson left a comment

Choose a reason for hiding this comment

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

LGTM. I diffed to generatedSuppressions and numbers for overlapping lines looks about right. Also spot checked a number to see they are still within the generatedSuppressions.

@chadlwilson
Copy link
Copy Markdown
Collaborator

Most of the stuff that is difficult to diff here is due to all the edits to the _project CPEs on base.

After this PR, we can perhaps remove the old ones from generatedSuppressions, since they have been correct in base for ~6 months via 12.1.3; there have been quite a few reasons for people to be forced to upgrade recently?

Alternatively we could sync them across to generatedSuppressions and then remove from base.

@jeremylong jeremylong merged commit cd45e1b into main Nov 20, 2025
5 checks passed
@jeremylong jeremylong deleted the scratch/cleanup-suppression-rules-2 branch November 20, 2025 13:07
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

core changes to core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants