Skip to content

Adding documentation for CA1516 - Use cross-platform intrinsics #47276

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 27, 2025

Conversation

@tannergooding tannergooding requested review from gewarren and a team 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
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