Skip to content

Launch file cleanup#1380

Merged
AndyZe merged 2 commits intomoveit:mainfrom
AndyZe:andyz/ros2_control_launching
Jun 20, 2022
Merged

Launch file cleanup#1380
AndyZe merged 2 commits intomoveit:mainfrom
AndyZe:andyz/ros2_control_launching

Conversation

@AndyZe
Copy link
Copy Markdown
Member

@AndyZe AndyZe commented Jun 19, 2022

Description

Clean up several launch files:

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 19, 2022

Codecov Report

Merging #1380 (e6bf16f) into main (491f1c8) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1380      +/-   ##
==========================================
- Coverage   61.56%   61.52%   -0.04%     
==========================================
  Files         274      274              
  Lines       24977    24977              
==========================================
- Hits        15375    15364      -11     
- Misses       9602     9613      +11     
Impacted Files Coverage Δ
moveit_ros/moveit_servo/src/servo_calcs.cpp 65.06% <0.00%> (-1.16%) ⬇️
...nning_scene_monitor/src/planning_scene_monitor.cpp 45.29% <0.00%> (-0.43%) ⬇️

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 491f1c8...e6bf16f. Read the comment docs.

Copy link
Copy Markdown
Contributor

@vatanaksoytezer vatanaksoytezer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you for doing this @AndyZe!

@AndyZe AndyZe merged commit 038cb20 into moveit:main Jun 20, 2022
peterdavidfagan pushed a commit to peterdavidfagan/moveit2 that referenced this pull request Jul 14, 2022
* Launch file cleanup

* Delete deprecated launch files
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.

2 participants