-
Notifications
You must be signed in to change notification settings - Fork 7
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
Formatting #35
Comments
Agreed, how easy is it to add the action? |
I actually meant something akin to this action, which runs clang-format and clang-tidy checks. Taking it a step further, we could each setup pre-commit hooks on our systems which abort the commit if it does not conform to style guide (this one aborts and patches with the fixes automatically). I'm sure this could be written server-side as well if we really wanted. |
If we do any linting checks we will want to have a formatter plugin that enforces that formatting. |
Our code is not keeping up with the Google/Drake style guide. We should do a series of formatting commits to bring things up to spec and create a
ignore-revs-file
file to keepblame
s honest. Also, might be worth considering a lint-on-push action.The text was updated successfully, but these errors were encountered: