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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move literals into resources #3026
Conversation
324295e
to
1624f4c
Compare
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.
@NextTurn Thanks for the attention to detail here. I noticed some other issues with the strings, but I don't want to make any of those changes in this large pull request. The only changes I was looking for are related directly to the changes in this pull request, including the specific comments.
StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1503BracesMustNotBeOmitted.cs
Outdated
Show resolved
Hide resolved
...Analyzers/StyleCop.Analyzers/LayoutRules/SA1507CodeMustNotContainMultipleBlankLinesInARow.cs
Outdated
Show resolved
Hide resolved
...rs/StyleCop.Analyzers/LayoutRules/SA1519BracesMustNotBeOmittedFromMultiLineChildStatement.cs
Outdated
Show resolved
Hide resolved
StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1520UseBracesConsistently.cs
Outdated
Show resolved
Hide resolved
...p.Analyzers/StyleCop.Analyzers/NamingRules/SX1309SStaticFieldNamesMustBeginWithUnderscore.cs
Show resolved
Hide resolved
StyleCop.Analyzers/StyleCop.Analyzers/NamingRules/SX1309FieldNamesMustBeginWithUnderscore.cs
Show resolved
Hide resolved
...Cop.Analyzers/StyleCop.Analyzers/OrderingRules/SA1200UsingDirectivesMustBePlacedCorrectly.cs
Outdated
Show resolved
Hide resolved
...Cop.Analyzers/StyleCop.Analyzers/OrderingRules/SA1200UsingDirectivesMustBePlacedCorrectly.cs
Outdated
Show resolved
Hide resolved
298775d
to
2e7e9cd
Compare
Thank you ! 馃槃 |
Happy reviewing 馃槈