search

semihokur / AsyncFixer

Advanced async/await Diagnostics and CodeFixes for C#
Apache License 2.0
165 stars 11 forks source link

Documenting Mapping Between AsyncFixer and VSThreading Analyzers #36

Closed bbrandt closed 1 year ago

bbrandt commented 1 year ago

If you are using both Code Analyzer packages you will want to disable one or the other of these couple of duplicates.

AsyncFixer VSThreading
AsyncFixer03 VSTHRD100
AsyncFixer02 VSTHRD103

Closing immediately after opening this issue as I just don't know where else to document this.

bbrandt commented 1 year ago

This is what I have in my .editorconfig as a result:

# Debatable. Using async/await in the way described decreases performance, but has 
# the benefit of showing both calling and called method in the call stack.
dotnet_diagnostic.AsyncFixer01.severity = none

# Disabled.  Already implemented by VSTHRD103
dotnet_diagnostic.AsyncFixer02.severity = none

# Disabled. Already implemented by VSTHRD100
dotnet_diagnostic.AsyncFixer03.severity = none

What I am not sure about is if AsyncFixer04 and AsyncFixer05 are implemented by VSThreading's analyzer.