search

DotNetAnalyzers / AsyncUsageAnalyzers

Now superseded by Microsoft/vs-threading
https://github.com/Microsoft/vs-threading
Other
121 stars 18 forks source link

Support Supression attribute #45

Closed dersteve closed 8 years ago

dersteve commented 8 years ago

At the moment there is only support for "pragma" for supression of a rule.

It would be good to be consistent and support SupressionAttribute as well.

sharwell commented 8 years ago

If SuppressMessage is not working, you should probably file a bug in the dotnet/roslyn repository, since suppressions are handled entirely by the compiler itself. Do you have a specific example (code) where we can see this failing?

dersteve commented 8 years ago

I just assumed it would not be working, because here [1] it talks only about the 'pragma approach'

[1] https://github.com/DotNetAnalyzers/AsyncUsageAnalyzers/blob/master/documentation/UseAsyncSuffix.md

sharwell commented 8 years ago

I filed #46 to get this addressed. :smile: