Skip to content

CppLint: Make "root" flag configurable via CPPLINT.cfg#129

Merged
eglaysher merged 1 commit intogoogle:gh-pagesfrom
gueraf:gh-pages
Jun 30, 2016
Merged

CppLint: Make "root" flag configurable via CPPLINT.cfg#129
eglaysher merged 1 commit intogoogle:gh-pagesfrom
gueraf:gh-pages

Conversation

@gueraf
Copy link
Contributor

@gueraf gueraf commented May 1, 2016

Enables CppLint to pick up the "root" flag from a CPPLINT.cfg file.

Practical use: The CppStyle plugin for Eclipse, for example, doesn't allow passing flags to CppLint. Using CPPLINT.cfg files comes in handy here.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@gueraf
Copy link
Contributor Author

gueraf commented May 1, 2016

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

tkruse pushed a commit to tkruse/cpplint that referenced this pull request May 12, 2016
@tkruse
Copy link

tkruse commented May 12, 2016

I added this PR to my cpplint fork as tkruse/cpplint#30

tkruse added a commit to tkruse/styleguide that referenced this pull request May 13, 2016
(copied from google#129)

# Conflicts:
#	cpplint.py
@wimrijnders
Copy link

wimrijnders commented May 19, 2016

Thanks, guys! I'll take it for a spin again as soon as I can get the change.

@eglaysher
Copy link
Contributor

I do apologize for the delay in getting to this; previously, no one was watching this repository. This looks straightforward enough. Thank you for your contribution.

@eglaysher eglaysher merged commit 01f0f70 into google:gh-pages Jun 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants