-
Notifications
You must be signed in to change notification settings - Fork 938
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
moveit_cpp: Cover another failure path in planning pipeline loading #2336
moveit_cpp: Cover another failure path in planning pipeline loading #2336
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2336 +/- ##
==========================================
- Coverage 56.73% 56.73% -0.00%
==========================================
Files 283 283
Lines 25455 25457 +2
==========================================
+ Hits 14440 14441 +1
- Misses 11015 11016 +1
Continue to review full report at Codecov.
|
…ailure condition.
Accidentally closed this by pushing an empty changeset |
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.
More helpful error messages are always nice. Thank you.
…ailure condition. (moveit#2336)
…ailure condition. (moveit#2336)
Add a more informative error message, and cover another potential failure condition.