uwreact / uwreact_robot

Software behind our fully autonomous FIRST robots

BSD 3-Clause "New" or "Revised" License

8 stars 6 forks source link

CI style checks #79

Closed matthew-reynolds closed 5 years ago

matthew-reynolds commented 5 years ago

Pull Request

Closes #57, duplicating changes from uwreact/frc_control#59.

Adds catkin, C++, python, and generic linting/formatting checks to the CI configuration to ensure we only commit quality code. More details can be found in the corresponding PR in frc_control.

Note that there are a couple small bigfixes in this PR not present on uwreact/frc_control#59. These will be duplicated back over to frc_control.

Contribution Checklist

[x] I have read the contributing guide.
[x] I have referenced relevant issues.
[x] I have labeled accordingly.
[x] I have detailed my changes as much as possible.

Change Checklist

[x] I have added tests.
[x] I have added necessary documentation.
[x] I have auto formatted using clang-format or yapf.
[ ] I have tested my changes on real hardware.