Skip to content
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

Close YAML file when we're done. #415

Merged
merged 6 commits into from
Jul 1, 2020
Merged

Conversation

rotu
Copy link
Contributor

@rotu rotu commented May 6, 2020

Signed-off-by: Dan Rose [email protected]

@hidmic
Copy link
Contributor

hidmic commented May 14, 2020

@rotu ping.

@ivanpauno ivanpauno mentioned this pull request May 14, 2020
@rotu rotu marked this pull request as draft May 14, 2020 15:19
@rotu
Copy link
Contributor Author

rotu commented May 14, 2020

converting to draft to wait until #414 is done

@rotu rotu force-pushed the close_yaml_file branch from 099d9c9 to af011b6 Compare May 28, 2020 22:28
@rotu rotu force-pushed the close_yaml_file branch 2 times, most recently from 9c39fae to 8e0e317 Compare June 15, 2020 19:42
@ivanpauno
Copy link
Member

@rotu friendly ping

@rotu rotu force-pushed the close_yaml_file branch from 7cccf99 to 794ee65 Compare June 30, 2020 23:58
rotu added 3 commits June 30, 2020 19:03
@rotu rotu force-pushed the close_yaml_file branch from 794ee65 to 9b18a21 Compare July 1, 2020 00:04
@rotu rotu marked this pull request as ready for review July 1, 2020 00:04
@rotu
Copy link
Contributor Author

rotu commented Jul 1, 2020

Thanks, @ivanpauno

Signed-off-by: Dan Rose <[email protected]>
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

launch_yaml/launch_yaml/parser.py Outdated Show resolved Hide resolved
Signed-off-by: Dan Rose <[email protected]>
@ivanpauno
Copy link
Member

Rci job is passing, merging!
Thanks for the fix @rotu!

@ivanpauno ivanpauno merged commit 85a7435 into ros2:master Jul 1, 2020
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