-
Notifications
You must be signed in to change notification settings - Fork 493
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
Update .pre-commit-config.yaml #2382
Conversation
Updated vodespell
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2382 +/- ##
=======================================
Coverage 50.84% 50.84%
=======================================
Files 386 386
Lines 31938 31938
=======================================
Hits 16237 16237
Misses 15701 15701 ☔ View full report in Codecov by Sentry. |
Looks like the new version is catching a bunch of new spelling issues. |
Here's a link to the issues found by the format check: https://github.com/ros-planning/moveit2/actions/runs/6453757742/job/17517921805?pr=2382#step:5:181 |
This has already been fixed by #2465 Just a suggestion, usually a better way to update pre-commit is to run |
Thanks @mosfet80 and @Shobuj-Paul! |
Updated codespell
Description
Please explain the changes you made, including a reference to the related issue if applicable
Checklist