Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Dec 11, 2025

Description

Add XML/YAML launch file equivalents "Using ROS 2 Launch For Large Projects" tutorial.

  • Add XML and YAML equivalents for all Python launch files
  • Update tutorial with tabbed multi-format code examples

Fixes #6006

Did you use Generative AI?

Partially - Cursor AI Assistant for tutorial rephrasing and refactoring. Each generated line reviewed.

Additional Information

  • broadcast_listener_launch.py renamed to broadcaster_listener_launch.py for correctness for launch file name
  • Only made changes to Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.html. If 📝 Replace all trivial launchfile examples with XML/YAML #6006 includes changes to tf and urdf tutorial, they will be done so in a separate PR.

This is an automatic backport of pull request #6021 done by [Mergify](https://mergify.com).

Signed-off-by: Luke Sy <[email protected]>
Signed-off-by: Katherine Scott <[email protected]>
Co-authored-by: Katherine Scott <[email protected]>
(cherry picked from commit 536c723)

# Conflicts:
#	source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
@mergify
Copy link
Contributor Author

mergify bot commented Dec 11, 2025

Cherry-pick of 536c723 has failed:

On branch mergify/bp/humble/pr-6021
Your branch is up to date with 'origin/humble'.

You are currently cherry-picking commit 536c723.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	renamed:    source/Tutorials/Intermediate/Launch/launch/broadcast_listener_launch.py -> source/Tutorials/Intermediate/Launch/launch/broadcaster_listener_launch.py
	new file:   source/Tutorials/Intermediate/Launch/launch/broadcaster_listener_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/broadcaster_listener_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/fixed_broadcaster_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/fixed_broadcaster_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/launch_turtlesim_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/launch_turtlesim_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/mimic_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/mimic_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_rviz_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_rviz_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_world_1_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_world_1_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_world_2_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_world_2_launch.yaml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_world_3_launch.xml
	new file:   source/Tutorials/Intermediate/Launch/launch/turtlesim_world_3_launch.yaml

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify bot added the conflicts label Dec 11, 2025
Copy link
Collaborator

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

@lsy3 do you have some time to review the conflicts ?

@lsy3
Copy link
Contributor

lsy3 commented Dec 11, 2025

@ahcorde The changes are complete and reflected in PR #6091 . I am not exactly sure how to push commits to this branch.

Remove conflict markers, keep multi-format references, and fix
PushRosNamespace capitalization.

Signed-off-by: Luke Sy <[email protected]>
@github-actions
Copy link

github-actions bot commented Dec 12, 2025

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/20162042847/artifacts/4848429007.

To view the resulting site:

  1. Click on the above link to download the artifacts archive
  2. Extract it
  3. Open html-artifacts-6087/index.html in your favorite browser

@ahcorde ahcorde enabled auto-merge (squash) December 12, 2025 09:16
@ahcorde ahcorde requested a review from fujitatomoya December 12, 2025 09:24
@ahcorde
Copy link
Collaborator

ahcorde commented Dec 12, 2025

@kscottz or @fujitatomoya can you merge this one ?

@mjcarroll mjcarroll disabled auto-merge December 12, 2025 14:48
@mjcarroll mjcarroll merged commit e74744a into humble Dec 12, 2025
5 checks passed
@mjcarroll mjcarroll deleted the mergify/bp/humble/pr-6021 branch December 12, 2025 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants