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

Add a new SuppressMessageAttribute target scope to suppress diagnosti… #31092

Merged

Commits on Nov 10, 2018

  1. Add a new SuppressMessageAttribute target scope to suppress diagnosti…

    …cs in a namespace and all its descendant symbols
    
    Fixes dotnet#486
    
    This SuppressMessageAttribute feature has been requested for a very long time and still gets frequent customer requests to date.
    
    The current "Namespace" target scope for SuppressMessageAttribute suppresses diagnostics only in nodes directly contained within the namespace, but in none of it's child symbols. This makes it very tedious to suppress certain analyzer diagnostics in entire namespace as one needs to add suppressions to individual types/namespaces within the namespace. This also creates a maintenance nightmare.
    
    This PR adds a new target scope "NamespaceAndChildren" which suppresses diagnostics on the corresponding namespace and all its descendant symbols/members. Better naming suggestions for this scope name are welcome!
    mavasani committed Nov 10, 2018
    Configuration menu
    Copy the full SHA
    677d520 View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2018

  1. Address feedback

    mavasani committed Nov 14, 2018
    1 Configuration menu
    Copy the full SHA
    83b90db View commit details
    Browse the repository at this point in the history