Skip to content
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

Add error_code_to_string function #1523

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

sjahr
Copy link
Contributor

@sjahr sjahr commented Aug 17, 2022

Description

Adds convenience function to print a MoveItError code

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

@sjahr sjahr requested a review from tylerjw August 17, 2022 15:55
@codecov
Copy link

codecov bot commented Aug 17, 2022

Codecov Report

Merging #1523 (f8c0a4e) into main (f7655df) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1523      +/-   ##
==========================================
+ Coverage   50.97%   50.98%   +0.01%     
==========================================
  Files         380      380              
  Lines       31758    31758              
==========================================
+ Hits        16187    16188       +1     
+ Misses      15571    15570       -1     
Impacted Files Coverage Δ
...dl_kinematics_plugin/src/kdl_kinematics_plugin.cpp 75.36% <0.00%> (-1.07%) ⬇️
...nning_scene_monitor/src/planning_scene_monitor.cpp 45.73% <0.00%> (+0.44%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tylerjw tylerjw force-pushed the pr-add_error_code_to_string branch from 8f1fae7 to cb06b30 Compare August 23, 2022 14:27
@sjahr sjahr force-pushed the pr-add_error_code_to_string branch from cb06b30 to f8c0a4e Compare August 24, 2022 14:46
@vatanaksoytezer vatanaksoytezer merged commit 042709a into moveit:main Aug 24, 2022
@tylerjw tylerjw added the backport-humble Mergify label that triggers a PR backport to Humble label Oct 28, 2022
mergify bot pushed a commit that referenced this pull request Oct 28, 2022
tylerjw pushed a commit that referenced this pull request Oct 28, 2022
(cherry picked from commit 042709a)

Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble Mergify label that triggers a PR backport to Humble
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants