-
Notifications
You must be signed in to change notification settings - Fork 182
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
Fix multi-line strings in DeclareLaunchArgument #948
Fix multi-line strings in DeclareLaunchArgument #948
Conversation
ba9689a
to
d623a4f
Compare
Can you please provide a short justification for why this is better? Both versions seem to work equally well for me, and e.g. |
Sorry for not provinding a motivation. This removes the whitespace from the argument descriptions. (Another option is removing the indent from the 2nd, 3rd, etc line. But I think this is better.) |
@RobertWilbrandt friendly ping |
1 similar comment
@RobertWilbrandt friendly ping |
d623a4f
to
3937e3c
Compare
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.
This looks good. Thank you @MatthijsBurgh for taking the time to implement this.
* (driver) fix multi-line strings in DeclareLaunchArgument * (moveit) fix multi-line strings in DeclareLaunchArgument (cherry picked from commit d16eaba) # Conflicts: # ur_robot_driver/launch/ur10.launch.py # ur_robot_driver/launch/ur10e.launch.py # ur_robot_driver/launch/ur16e.launch.py # ur_robot_driver/launch/ur20.launch.py # ur_robot_driver/launch/ur3.launch.py # ur_robot_driver/launch/ur30.launch.py # ur_robot_driver/launch/ur3e.launch.py # ur_robot_driver/launch/ur5.launch.py # ur_robot_driver/launch/ur5e.launch.py # ur_robot_driver/launch/ur_control.launch.py
* (driver) fix multi-line strings in DeclareLaunchArgument * (moveit) fix multi-line strings in DeclareLaunchArgument (cherry picked from commit d16eaba) # Conflicts: # ur_robot_driver/launch/ur10.launch.py # ur_robot_driver/launch/ur10e.launch.py # ur_robot_driver/launch/ur16e.launch.py # ur_robot_driver/launch/ur20.launch.py # ur_robot_driver/launch/ur3.launch.py # ur_robot_driver/launch/ur30.launch.py # ur_robot_driver/launch/ur3e.launch.py # ur_robot_driver/launch/ur5.launch.py # ur_robot_driver/launch/ur5e.launch.py # ur_robot_driver/launch/ur_control.launch.py
* (driver) fix multi-line strings in DeclareLaunchArgument * (moveit) fix multi-line strings in DeclareLaunchArgument (cherry picked from commit d16eaba)
* (driver) fix multi-line strings in DeclareLaunchArgument * (moveit) fix multi-line strings in DeclareLaunchArgument (cherry picked from commit d16eaba)
Related to:
UniversalRobots/Universal_Robots_ROS2_Description#140
UniversalRobots/Universal_Robots_ROS2_Gazebo_Simulation#59
UniversalRobots/Universal_Robots_ROS2_GZ_Simulation#16