Action to add file-level ignore on violation

[zanieb]

A quick fix for a violation shows an option to "Ignore the violation on this line", I'd like the additional option to "Ignore the violation in this file" which would be useful since I don't necessarily remember the syntax to do so :)

[charliermarsh]

My only concern would be that we're starting to accumulate a lot of quick fix actions 😂

[MichaReiser]

My only concern would be that we're starting to accumulate a lot of quick fix actions 😂

That, and I don't think suppressing an entire violation for an entire file is something we want to encourage. How do other linter solve this? Could we add a command instead?

[zanieb]

I think it makes a lot more sense as a code-action than a command. A command feels like we're encouraging it more i.e. you can apply it to multiple files at once. A code-action is like.. do this manually once.

I don't think suppressing an entire violation for an entire file is something we want to encourage

I guess the other option is to suggest using per-file-ignores instead? I'm not sure that's really better though.

[dhruvmanila]

FWIW, Lua language server provides 3 options (line, file, workspace):