AtomLinter / linter-cpplint

Linter plugin for c++, using cpplint
https://atom.io/packages/linter-cpplint
MIT License
6 stars 5 forks source link

Load Configurations from CPPLINT.cfg #61

Open Danielmelody opened 8 years ago

Danielmelody commented 8 years ago

As the documentation said,

cpplint.py supports per-directory configurations specified in CPPLINT.cfg files. CPPLINT.cfg file can contain a number of key=value pairs. Currently the following options are supported:

  set noparent
  filter=+filter1,-filter2,...
  exclude_files=regex
  linelength=80
  root=subdir
  headers=x,y,...

Is there any opportunity to do that?

cwoodall commented 7 years ago

This would be great!

Arcanemagus commented 7 years ago

This might be fixed by https://github.com/AtomLinter/linter-cpplint/pull/67, or even possibly by moving away from the outdated bundled version (https://github.com/AtomLinter/linter-cpplint/pull/66). Either way can you guys try v2.0.0 and see if it works for you?

Please note that as this package no longer bundles an out of date cpplint, you will need to have it available in your $PATH (or specify the path in the settings).

gvoigtlaender commented 5 years ago

I can confirm: having a CPPLINT.cfg in project root directory works for me with with 2.1.0. At least the header_guard respects the "root=." entry