Skip to content

Conversation

@tannergooding tannergooding requested review from a team and gewarren as code owners July 9, 2025 21:46
@dotnetrepoman dotnetrepoman bot added this to the July 2025 milestone Jul 9, 2025
Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

Thanks @tannergooding. Can you add it the new rule to the various index and TOC files (you can see an example here)? Also, can you add CA1516 to the output_kind option table?

@tannergooding
Copy link
Member Author

@gewarren, done!

Still waiting on the actual analyzer to finish being reviewed and merged.

@gewarren gewarren added the 🚧 Hold for related PR Indicates a PR can only be merged when other related PRs are merged (see comments for links) label Jul 22, 2025
@tannergooding
Copy link
Member Author

Analyzer was merged, so this should be good to merge as well now

@gewarren gewarren removed the 🚧 Hold for related PR Indicates a PR can only be merged when other related PRs are merged (see comments for links) label Jul 27, 2025
@gewarren gewarren enabled auto-merge (squash) July 27, 2025 03:18
@gewarren gewarren merged commit 0ba2191 into dotnet:main Jul 27, 2025
10 checks passed
@tannergooding tannergooding deleted the ca1516 branch July 27, 2025 04:07
aljam3h referenced this pull request Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants