Skip to content

Add wildcard loading to ros2 param load#602

Merged
ivanpauno merged 4 commits intoros2:masterfrom
pal-robotics-forks:add-load-wildcard-support
Feb 23, 2021
Merged

Add wildcard loading to ros2 param load#602
ivanpauno merged 4 commits intoros2:masterfrom
pal-robotics-forks:add-load-wildcard-support

Conversation

@v-lopez
Copy link
Contributor

@v-lopez v-lopez commented Feb 23, 2021

Related to #590 and #596

Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
@v-lopez v-lopez mentioned this pull request Feb 23, 2021
@ivanpauno ivanpauno added the enhancement New feature or request label Feb 23, 2021
@v-lopez v-lopez force-pushed the add-load-wildcard-support branch from 5bc06e6 to 20f56cf Compare February 23, 2021 13:46
v-lopez and others added 2 commits February 23, 2021 14:47
Co-authored-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
@v-lopez v-lopez force-pushed the add-load-wildcard-support branch from 20f56cf to 4212449 Compare February 23, 2021 13:47
Copy link
Member

@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the PR @v-lopez !!

@ivanpauno
Copy link
Member

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno
Copy link
Member

@v-lopez there are a few linter failures

Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
@ivanpauno
Copy link
Member

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno ivanpauno merged commit 4848995 into ros2:master Feb 23, 2021
v-lopez added a commit to pal-robotics-forks/ros2cli that referenced this pull request Feb 23, 2021
Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
ivanpauno pushed a commit that referenced this pull request Mar 5, 2021
Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants