Skip to content

fix: Add standard ROS default globs for input file discovery#6000

Merged
ruben-arts merged 2 commits intoprefix-dev:mainfrom
rshanor:add-ros-default-globs
May 4, 2026
Merged

fix: Add standard ROS default globs for input file discovery#6000
ruben-arts merged 2 commits intoprefix-dev:mainfrom
rshanor:add-ros-default-globs

Conversation

@rshanor
Copy link
Copy Markdown
Contributor

@rshanor rshanor commented May 3, 2026

Description

Add launch files, config, messages, services, and actions to the default input globs so they don't need to be passed via extra-input-globs. Update default input globs documentation. Previously briefly discussed with @Tobias-Fischer on discord.

How Has This Been Tested?

Ran all unit tests locally with cargo test -p pixi-build-ros

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.

Tools: Claude

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation

rshanor added 2 commits May 2, 2026 20:35
Add launch files, config, messages, services, and actions to the
default input globs so they don't need to be passed via extra-input-globs.
Update the extra-input-globs section to reflect the new default globs
added for launch files, config, messages, services, and actions.
@rshanor rshanor changed the title Add standard ROS default globs for input file discovery fix: Add standard ROS default globs for input file discovery May 3, 2026
@baszalmstra baszalmstra requested a review from ruben-arts May 3, 2026 05:19
Copy link
Copy Markdown
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

Perfect thank you!

@ruben-arts ruben-arts merged commit e2149e3 into prefix-dev:main May 4, 2026
40 of 41 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.

3 participants