Skip to content

Add tip for concatenating multiple (Python) substitutions#5157

Merged
christophebedard merged 1 commit intorollingfrom
christophebedard/document-launch-py-substitution-concatenation
Apr 22, 2025
Merged

Add tip for concatenating multiple (Python) substitutions#5157
christophebedard merged 1 commit intorollingfrom
christophebedard/document-launch-py-substitution-concatenation

Conversation

@christophebedard
Copy link
Copy Markdown
Member

@christophebedard christophebedard commented Mar 23, 2025

This aims to resolve ros2/launch#835

This explicitly mentions the usual substitution concatenation behaviour. I think this is a good tutorial and section for this, but it only applies to Python launch files, so it only made sense to make it a .. tip:: block and not a standalone (sub)section.

The example depends on ros2/launch#838.

Note

This is currently only available in Rolling (including Kilted soon), so this PR shouldn't be backported.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 23, 2025

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/14563913021/artifacts/2976733574.

To view the resulting site:

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

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@christophebedard christophebedard marked this pull request as ready for review April 20, 2025 22:23
@christophebedard christophebedard force-pushed the christophebedard/document-launch-py-substitution-concatenation branch from 59612cd to 22237ba Compare April 20, 2025 22:25
@christophebedard
Copy link
Copy Markdown
Member Author

This is ready for review.

@christophebedard christophebedard force-pushed the christophebedard/document-launch-py-substitution-concatenation branch from 22237ba to 4dfd427 Compare April 20, 2025 22:30
Copy link
Copy Markdown
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

lgtm

@christophebedard christophebedard merged commit 292e42c into rolling Apr 22, 2025
5 checks passed
@christophebedard christophebedard deleted the christophebedard/document-launch-py-substitution-concatenation branch April 22, 2025 16:08
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.

Document how multiple substitutions can be concatenated into a single string using a list

2 participants