Skip to content

Conversation

@jeremylong
Copy link
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
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
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
9 checks passed
@jeremylong jeremylong deleted the scratch/cleanup-suppression-rules-2 branch November 20, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core changes to core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants