-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update doc for CA1305 to mention excluded types #5016
Conversation
@Evangelink : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
@@ -35,6 +35,9 @@ This rule ignores calls to .NET methods that are documented as ignoring the <xre | |||
- <xref:System.Activator.CreateInstance%2A?displayProperty=nameWithType> | |||
- <xref:System.Resources.ResourceManager.GetObject%2A?displayProperty=nameWithType> | |||
- <xref:System.Resources.ResourceManager.GetString%2A?displayProperty=nameWithType> | |||
- <xref:bool.ToString%2A?displayProperty=nameWithType> | |||
- <xref:char.ToString%2A?displayProperty=nameWithType> | |||
- <xref:System.Guid.ToString%2A?displayProperty=nameWithType> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Evangelink Can you remind me of the PR on roslyn-analyzers to fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mavasani The changes here look OK to merge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes.
#sign-off Thanks for the doc updates, @Evangelink ! |
Fix #4986
This new feature is not yet publicly released. Please sync-up with @mavasani for when to merge this PR.