On Mon, Oct 29, 2018 at 09:39:33AM +0900, Junio C Hamano wrote: > "brian m. carlson" <sandals@xxxxxxxxxxxxxxxxxxxx> writes: > > This may not strictly be needed, but removing it makes the header no > > longer self-contained, which blows up my (and others') in-editor > > linting. > > That sounds like bending backwards to please tools, though. Can't > these in-editor linting learn the local rules of the codebase they > are asked to operate on? Doing so involves writing (in my case) Vim code to configure settings for this repo. Since language linting tools invoke compilers and other language runtimes, you need to specify command-line arguments to those tools, and in general, that's not a safe thing you can read from the repository configuration, since just viewing files should not be able to execute arbitrary code[0]. Languages such as Perl which can execute arbitrary code with compile checks have to be enabled explicitly with ALE (which is what I'm using). I need to do a reroll to address some other issues people brought up, so I can remove this line. [0] Pass "-o .bashrc" to the preprocessor, for example. -- brian m. carlson: Houston, Texas, US OpenPGP: https://keybase.io/bk2204
Attachment:
signature.asc
Description: PGP signature
