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

Small clang-tidy fix #3129

Merged
merged 1 commit into from
May 5, 2022
Merged

Small clang-tidy fix #3129

merged 1 commit into from
May 5, 2022

Conversation

rhaschke
Copy link
Contributor

@rhaschke rhaschke commented May 4, 2022

No description provided.

@rhaschke rhaschke force-pushed the fix-clang-tidy branch 2 times, most recently from 141ab7e to b2d640c Compare May 5, 2022 10:27
@rhaschke rhaschke requested a review from v4hn as a code owner May 5, 2022 10:33
@codecov
Copy link

codecov bot commented May 5, 2022

Codecov Report

Merging #3129 (530afac) into master (bb9a767) will increase coverage by 0.07%.
The diff coverage is 0.00%.

❗ Current head 530afac differs from pull request most recent head f326984. Consider uploading reports for the commit f326984 to get more accurate results

@@            Coverage Diff             @@
##           master    #3129      +/-   ##
==========================================
+ Coverage   61.58%   61.65%   +0.07%     
==========================================
  Files         376      376              
  Lines       33311    33309       -2     
==========================================
+ Hits        20512    20533      +21     
+ Misses      12799    12776      -23     
Impacted Files Coverage Δ
...llision_detection/src/collision_octomap_filter.cpp 0.00% <0.00%> (ø)
...eit_ros/manipulation/pick_place/src/pick_place.cpp 88.58% <0.00%> (-3.80%) ⬇️
...ipulation/pick_place/src/manipulation_pipeline.cpp 73.56% <0.00%> (-0.82%) ⬇️
...nning_scene_monitor/src/planning_scene_monitor.cpp 66.47% <0.00%> (+0.13%) ⬆️
...raint_samplers/src/default_constraint_samplers.cpp 79.71% <0.00%> (+0.30%) ⬆️
...meterization/work_space/pose_model_state_space.cpp 81.77% <0.00%> (+1.77%) ⬆️
moveit_core/robot_model/src/joint_model_group.cpp 64.15% <0.00%> (+2.28%) ⬆️
...e/src/parameterization/model_based_state_space.cpp 71.43% <0.00%> (+3.11%) ⬆️
.../ompl_interface/src/detail/constrained_sampler.cpp 60.98% <0.00%> (+17.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c777b7...f326984. Read the comment docs.

@v4hn v4hn merged commit 0f66b99 into moveit:master May 5, 2022
@rhaschke rhaschke deleted the fix-clang-tidy branch May 5, 2022 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants