Fail on no analyzers registered

ionide / FSharp.Analyzers.SDK

Library for building custom analyzers for F# / FSAC

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

MIT License

74 stars 21 forks source link

Fail on no analyzers registered #202

Closed Smaug123 closed 7 months ago

Smaug123 commented 7 months ago

(I seem to be hitting every possible way to hold this tool wrongly!)

I think it's more useful to fail when there are no analyzers registered, than to return "OK, did nothing". I suspect it's going to be very unusual to intentionally register no analyzers.

nojaf commented 7 months ago

@dawedawe if you agree, can you approve and merge?

Smaug123 commented 7 months ago

(No need to release this any time soon.)

nojaf commented 7 months ago

Sweet 😌