-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
[Analyzers] Resolve StyleCop issues: SA1516 and SA1616 #34853
Conversation
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.
all whitespace, good to go
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.
Hi @snickler , Thanks for opening the PR.
Why is this PR creating a AvancedPasteContext.cs file? e450163#diff-4cc7d732bfa287b2b2592df644079740a8c18868be1a6c57e0121a2d573ea6ac
Thought this was only whitespace fixes that were needed.
Ah thanks for finding that. I accidentally committed the fileI have for ANOTHER thing I'm working on ;). Let me fix that since it's not supposed to be checked in. |
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.
LGTM! Thank you for the contribution!
Summary of the Pull Request
Resolved StyleCop issues SA1516 and SA1616 that have recently appeared in Visual Studio 17.12
PR Checklist
Detailed Description of the Pull Request / Additional comments
Validation Steps Performed