search

dense-analysis / ale

Check syntax in Vim/Neovim asynchronously and fix files, with Language Server Protocol (LSP) support
BSD 2-Clause "Simplified" License
13.4k stars 1.42k forks source link

checkpatch for C (kernel) #554

Open dawikur opened 7 years ago

dawikur commented 7 years ago

So kernel has this cool script checkpatch.pl to check if code follows the coding style. It would be very convenient to have it as a plugged linter. But I doubt kernel style should be default for all C projects. As I see, all linters are 'enabled' by default, would you be fine with disabled by linter? Is there already a functionality for this? I would be glad to create PR.

w0rp commented 7 years ago

The linter can be disabled by default by adding a List which lists every linter in it except the ones we don't want to enable by default.

dawikur commented 7 years ago

So you think it should be enabled by default and everyone who doesn't like it would need to disable it explicitly?

w0rp commented 7 years ago

That is one option. If the Perl script can be found, it could be enabled by default. Then it will only be enabled on systems where it is available. That's how most linters work.

I was suggesting disabling it by default by listing all of the other linters. That way, everything except checkpatch.pl will be enabled by default.

bhundven commented 2 years ago

Being worked on in #3912