search

microsoft / sarif-sdk

.NET code and supporting files for working with the 'Static Analysis Results Interchange Format' (SARIF, see https://github.com/oasis-tcs/sarif-spec)
Other
191 stars 88 forks source link

Better UX emitting `FailureLevel.Note` debug messages on the console. #2746

Closed michaelcfanning closed 7 months ago

michaelcfanning commented 7 months ago

Emit FailureLevel.Note messages with label info (rather than fail) in ConsoleLogger.

Previously, for debug (i.e., FailureLevel.Note messages) we emitted strings like:

temp.txt(1,1-108): fail TST101/016: '…xxxDid' is an apparent false positive (because it does not contain at least one digit and letter).

Now we emit fail as a more appropriate label for debug messages:

temp.txt(1,1-108): info TST101/016: '…xxxDid' is an apparent false positive (because it does not contain at least one digit and letter).