DotNetAnalyzers / AsyncUsageAnalyzers

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

Fix for UseAsyncSuffix should not fire for overriding methods (#33) #35

Closed ChrSteinert closed 8 years ago

ChrSteinert commented 9 years ago

This should fix #33 and #30 .

sharwell commented 8 years ago

It turns out this was only one piece of the puzzle. I implemented a more complete version of this in #39 along with unit tests that exercise each of the various paths.

I actually meant to merge this and then build on top of it, but after #38 it didn't work out cleanly. Thanks for taking the time to submit this though. :smile: