ionide / FSharp.Analyzers.SDK

Library for building custom analyzers for F# / FSAC

Home Page:http://ionide.io/FSharp.Analyzers.SDK/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Cli option --fail-on-all-warnings

dawedawe opened this issue · comments

commented

Is your feature request related to a problem? Please describe.
Currently, to fail on a warning, you have to list all the analyzer codes you want to fail on. If you are using many analyzers and you want to fail on all of them, that can be cumbersome.

Describe the solution you'd like
Provide a --fail-on-all-warnings Cli option.

Describe alternatives you've considered
Support globbing of analyzer codes.

Additional context
As always, I just raise this issue to keep track of it.

Consider a syntax like NoWarn in the compiler to allow setting diagnostic levels on a per message basis, as well as the more general 'treatwarningsaserror' mechanism: https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-options/errors-warnings

I think it would be good for the analyzer SDK to onboard to these mechanisms because that's how they would be handled/integrated in the 'final' analyzer setup, when they're part of the compiler.