cpplint 2.x has been released, with new rules and improvments to existing rules causing our lint CI to now fail.
From a glance at the failures we should:
[ ] Prevent the being banned rule via CPPLINT.cfg, filesystem is useful.
[ ] Prevent the intend in namespace rule via CPPLINT.cfg
[ ] Probably fix the new include what you use errors
This will need applying to many repositories.
We could also pin to cpplint 1.x instead, but we'd also have to ammend our readme's to tell users its only 1.x compatible.
Alternatively we could switch to clang-format and see if it's any more stable (buit that would take more time and effort and installation used to be more of a faff than cpplint)
cpplint 2.x has been released, with new rules and improvments to existing rules causing our lint CI to now fail.
From a glance at the failures we should:
This will need applying to many repositories.
We could also pin to cpplint 1.x instead, but we'd also have to ammend our readme's to tell users its only 1.x compatible.
Alternatively we could switch to clang-format and see if it's any more stable (buit that would take more time and effort and installation used to be more of a faff than cpplint)