Skip to content

[humble] Pull all example launchfiles into separate file with literalinclude (backport #5155) #5183

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

Merged
merged 2 commits into from
Mar 28, 2025

Conversation

emersonknapp
Copy link
Contributor

Replaces #5175

Resolves conflicts in that branch - without write access I can't update the Mergify bot PR, so I've fixed it here instead.

Backports #5155 to humble


Part of #5143

This way we can easily grab them for testing, manipulate programmatically, share without repeating (especially in block-by-block code walkthroughs), and maybe even run linters on them at some point to apply ROS 2 coding standards and maintain a consistent formatting.

I resisted the temptation to rewrite them, except for minor formatting fixes.

emersonknapp and others added 2 commits March 26, 2025 15:52
…ros2#5155)

Signed-off-by: Emerson Knapp <[email protected]>
(cherry picked from commit 666df3e)

# Conflicts:
#	source/How-To-Guides/Launch-file-different-formats.rst
#	source/Tutorials/Intermediate/Launch/Using-Event-Handlers.rst
#	source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
#	source/Tutorials/Intermediate/Launch/Using-Substitutions.rst
#	source/Tutorials/Intermediate/Tf2/Adding-A-Frame-Cpp.rst
#	source/Tutorials/Intermediate/Tf2/Adding-A-Frame-Py.rst
@emersonknapp
Copy link
Contributor Author

@christophebedard for review, perhaps

Copy link
Member

@christophebedard christophebedard left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for backporting it.

@christophebedard christophebedard merged commit e81381d into ros2:humble Mar 28, 2025
5 checks passed
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