Commit f3e104f0 authored by Philip Chimento's avatar Philip Chimento Committed by Philip Chimento

CI: Remove customizations from cpplint config

We're going to just use the Google C++ coding style, so remove our
customizations. (We do still allow lines a little bit longer than the
limit that we give to clang-format, since sometimes clang-format will
format a line longer than 80 characters if the alternative is worse.)

[skip cpplint] since this will of course introduce many more linter
errors.
parent 7b14e20e
......@@ -5,10 +5,13 @@ ColumnLimit: 80
---
Language: Cpp
BasedOnStyle: Google
AccessModifierOffset: -3 # to match cpplint
AllowShortIfStatementsOnASingleLine: false
AllowShortLoopsOnASingleLine: false
DerivePointerAlignment: false
ForEachMacros: []
IndentWidth: 4
PointerAlignment: Left # Google style allows both, but clang-format doesn't
---
# We rely mostly on eslint for JavaScript linting, but this is a lax collection
# of rules that will auto-fix some JS things. We really should use eslint --fix
......
# This is the toplevel CPPLINT.cfg file
set noparent
# We give a limit to clang-format of 80, but we allow 100 here for cases where
# it really is more readable to have a longer line
linelength=100
filter=-build/include_subdir
filter=-whitespace/braces
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment