Skip to content

parametric config path for macro - #416

Open
torydebra wants to merge 1 commit into
lbr-stack:jazzyfrom
torydebra:jazzy
Open

torydebra wants to merge 1 commit into
lbr-stack:jazzyfrom
torydebra:jazzy

Conversation

@torydebra

Copy link
Copy Markdown

Hello,

I just added a param in the xacro macro to define a custom path for the yaml configuration file

@mhubii

mhubii commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

hi @torydebra, thank you for the contribution, looks good to me. Would it be okay to target the PR against the rolling branch?

@mhubii mhubii self-assigned this Oct 6, 2026
@torydebra

Copy link
Copy Markdown
Author

Would it be okay to target the PR against the rolling branch?

Is this is meant for me? I think any branch is fine but I am currently working on jazzy so it would be nice to have it there as well

@mhubii

mhubii commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

yes yes, there is currently an action setup to backport, i.e. cherry-pick, from rolling to jazzy. Would be nice to have this change for future versions, and then the action will take care of making it available to jazzy.

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