Skip to content

Add FxCop scripts#18

Open
benmcmorran wants to merge 1 commit intodev15from
dev/bemcmorr/loc
Open

Add FxCop scripts#18
benmcmorran wants to merge 1 commit intodev15from
dev/bemcmorr/loc

Conversation

@benmcmorran
Copy link
Copy Markdown
Member

Two globalization rules are currently violated.

  1. CultureInfo.CurrentCulture is not always specified when formatting strings. This change appears to be low-priority, does not impact behavior, and may make the code noisier.
  2. In two cases, strings are converted to lowercase. In both instances, the strings are only used for output (test case results and a string summary of command line arguments), so we shouldn't run into issues with normalization. The FxCop file includes exclusions for both cases.

@benmcmorran benmcmorran requested a review from a team September 15, 2017 22:20
Copy link
Copy Markdown

@LukaszMendakiewicz LukaszMendakiewicz left a comment

Choose a reason for hiding this comment

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

Thanks. I guess this should now be squashed before merging.

@LukaszMendakiewicz
Copy link
Copy Markdown

Since this project switched to consuming NuGet through PackageReference for most of the packages (ideally should be for all, once we iron out some wrinkles around MicroBuild), does Flatten-NuGetPackages even make sense here, and don't the packages referenced through PackageReference pose any problems during analysis?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants