Skip to content

[foxy backport] Add pytest.ini to test_launch_testing so tests succeed locally. (#431)#451

Merged
jacobperron merged 2 commits intofoxyfrom
jacob/backport_431
Aug 18, 2020
Merged

[foxy backport] Add pytest.ini to test_launch_testing so tests succeed locally. (#431)#451
jacobperron merged 2 commits intofoxyfrom
jacob/backport_431

Conversation

@jacobperron
Copy link
Copy Markdown
Member

@jacobperron jacobperron commented Aug 13, 2020

Backport #431 to Foxy.

This should help resolve one of the outstanding failures in ros2/ci#503

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@jacobperron jacobperron changed the title Add pytest.ini to test_launch_testing so tests succeed locally. (#431) [foxy backport] Add pytest.ini to test_launch_testing so tests succeed locally. (#431) Aug 13, 2020
@jacobperron
Copy link
Copy Markdown
Member Author

@ros-pull-request-builder retest this please

@jacobperron
Copy link
Copy Markdown
Member Author

CI for this PR and several other similar backports can be found here: ros2/ci#503 (comment)

@jacobperron jacobperron changed the base branch from master to foxy August 17, 2020 21:59
Copy link
Copy Markdown
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

You are also going to need pytest.ini in the launch_xml, launch, and launch_yaml packages.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@jacobperron jacobperron merged commit f152bef into foxy Aug 18, 2020
@jacobperron jacobperron deleted the jacob/backport_431 branch August 18, 2020 16:42
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