Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add linter for missing "pragma once" #9516

Closed
jwnimmer-tri opened this issue Sep 25, 2018 · 1 comment
Closed

Add linter for missing "pragma once" #9516

jwnimmer-tri opened this issue Sep 25, 2018 · 1 comment

Comments

@jwnimmer-tri
Copy link
Collaborator

Every once in a while, a Drake PR or Drake-based PR is failing CI with a weird error, that we eventually track down to be a missing #pragma once in new (or worse, old!) code. It would be nice to update our cpplint.py to require the pragma.

@jwnimmer-tri
Copy link
Collaborator Author

I have a patch to a TRI fork of cpplint.py, that we could publish to RobotLocomotion/styleguide, if the platform review team likes the idea of enforcing this. What say you @EricCousineau-TRI @ggould-tri @sammy-tri @SeanCurtis-TRI @sherm1 @soonho-tri @RussTedrake? (Vote with sentiments on this post.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant