Skip to content
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

Configuration: Fix all warnings found when running Code Analysis with "Microsoft All Rules" #100

Closed
jcookems opened this issue Jul 10, 2012 · 1 comment

Comments

@jcookems
Copy link

Best practice is to review all warnings, and fix all unless there is a significant reason not to. Here are the current warnings from the dev branch:

  1. CA2210 : Microsoft.Design : Sign 'Microsoft.WindowsAzure.Configuration.dll' with a strong name key.
  2. CA1014 : Microsoft.Design : Mark 'Microsoft.WindowsAzure.Configuration.dll' with CLSCompliant(true) because it exposes externally visible types.
  3. CA1824 : Microsoft.Performance : Because assembly 'Microsoft.WindowsAzure.Configuration.dll' contains a ResX-based resource file, mark it with the NeutralResourcesLanguage attribute, specifying the language of the resources within the assembly. This could improve lookup performance the first time a resource is retrieved.
  4. CA1811 : Microsoft.Performance : 'AzureApplicationSettings.GetServiceSetting(string)' appears to have no upstream public or protected callers.
@ogail
Copy link
Contributor

ogail commented Oct 31, 2013

Fixed in 2.0.2 release

@ogail ogail closed this as completed Oct 31, 2013
hyonholee pushed a commit to hyonholee/azure-sdk-for-net that referenced this issue Sep 28, 2015
hovsepm pushed a commit that referenced this issue Apr 25, 2016
jethibau pushed a commit to jethibau/azure-sdk-for-net that referenced this issue Aug 2, 2018
@github-actions github-actions bot locked and limited conversation to collaborators Mar 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants