-
Notifications
You must be signed in to change notification settings - Fork 85
Add --namespace argument to launch command. #433
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
base: rolling
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, i would add the test but i do not see any tests for ros2 launch
...
@StefanFabian could you please follow up with adding tests? |
@StefanFabian, friendly ping |
I'm sorry, but we are preparing for a robotics competition in the second week of March, and I don't have any time to invest here until after the competition. |
d3b1ca1
to
0ea4d94
Compare
0e09968
to
35a832b
Compare
@kramer-sim was nice enough to add a test case. Does this work for you? @Yadunund @audrow |
@Mergifyio rebase |
…aunching a launch file. Signed-off-by: Stefan Fabian <[email protected]> Signed-off-by: Markus Kramer <[email protected]>
Signed-off-by: Markus Kramer <[email protected]>
✅ Branch has been successfully rebased |
Pulls: #433 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI is failing, could you address the failures?
|
||
|
||
if __name__ == '__main__': | ||
unittest.main() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing last line feed detected by github.
This PR adds support for pushing a namespace when launching a launch file.