-
Notifications
You must be signed in to change notification settings - Fork 52
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
Enforce prettier code styling #535
Conversation
92279c4
to
8a1d4f2
Compare
Codecov Report
@@ Coverage Diff @@
## master #535 +/- ##
==========================================
- Coverage 36.00% 35.76% -0.24%
==========================================
Files 1 1
Lines 150 151 +1
Branches 29 29
==========================================
Hits 54 54
- Misses 96 97 +1
Continue to review full report at Codecov.
|
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
8a1d4f2
to
a9c1268
Compare
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks good. Here's an approval - well at least "in spirit!"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgmt
Enforce linter style in the tests and note the autoformatting command in DEVELOPING.md.
Helps avoid the "please don't include formatting changes in functional PRs" comments. I think some people's editors, or maybe some pre-commit hook that I don't have is doing it for other developers.
Helps address comments on #523 and #524
Lint failure in first commit check https://github.com/ros-tooling/action-ros-ci/pull/535/checks?check_run_id=1884593227