Skip to content

Conversation

@bforget163
Copy link
Contributor

@bforget163 bforget163 commented Apr 17, 2025

Update ca2249.md and fix typo in the example code

The code example is incorrect the variable name should be str instead of string.


Internal previews

📄 File 🔗 Preview link
docs/fundamentals/code-analysis/quality-rules/ca2249.md CA2249: Consider using String.Contains instead of String.IndexOf

@bforget163 bforget163 requested review from a team and gewarren as code owners April 17, 2025 07:36
@dotnetrepoman dotnetrepoman bot added this to the April 2025 milestone Apr 17, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-fundamentals/svc community-contribution Indicates PR is created by someone from the .NET community. labels Apr 17, 2025
@bforget163
Copy link
Contributor Author

@dotnet-policy-service agree

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks for spotting this and fixing it @bforget163

I'll :shipit: now. You should see the changes on the live site in the next day or two on our regular publishing cycle. Thanks again.

@BillWagner BillWagner merged commit 8333f50 into dotnet:main Apr 17, 2025
11 checks passed
@bforget163 bforget163 deleted the patch-2 branch April 18, 2025 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-fundamentals/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants