Skip to content

Add docs for CA1859 analyzer rule#35182

Merged
geeknoid merged 1 commit into
dotnet:mainfrom
geeknoid:ca1859
May 2, 2023
Merged

Add docs for CA1859 analyzer rule#35182
geeknoid merged 1 commit into
dotnet:mainfrom
geeknoid:ca1859

Conversation

@geeknoid

@geeknoid geeknoid commented Apr 27, 2023

Copy link
Copy Markdown
Member

@Youssef1313 Youssef1313 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is missing a toc entry I believe

@geeknoid

Copy link
Copy Markdown
Member Author

This is missing a toc entry I believe

@Youssef1313 I searched for an existing rule number (CA1852 I think) and put references to 1859 in all the same spots. Which toc file do I need to add this to?

@gewarren gewarren left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/index.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/quality-rules/ca1859.md Outdated
Comment thread docs/fundamentals/code-analysis/code-quality-rule-options.md Outdated
@geeknoid geeknoid requested a review from a team as a code owner May 2, 2023 12:28
@geeknoid geeknoid merged commit cba7180 into dotnet:main May 2, 2023
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.

4 participants