Method name analyzers should not run run for override/implementing methods

DotNetAnalyzers / AsyncUsageAnalyzers

Now superseded by Microsoft/vs-threading

https://github.com/Microsoft/vs-threading

Other

121 stars 18 forks source link

Method name analyzers should not run run for override/implementing methods #30

Closed sharwell closed 8 years ago

sharwell commented 9 years ago

Currently the UseAsyncSuffix diagnostic is reported for cases where a method overrides a method from a base class and/or implements an interface method. In these cases, the name is set by the base definition, so no naming warnings should be reported at the site of an overriding or implementing method.

ChrSteinert commented 9 years ago

Isn't this duplicated by #33 ?

ChrSteinert commented 9 years ago

On second thought - this is also for AvoidAsyncSuffixAnalyzer isn't it?