Skip to content

[foxy backport] Add in pytest.ini so tests succeed locally. (#280)#309

Merged
jacobperron merged 3 commits intofoxyfrom
jacob/backport_280
Aug 18, 2020
Merged

[foxy backport] Add in pytest.ini so tests succeed locally. (#280)#309
jacobperron merged 3 commits intofoxyfrom
jacob/backport_280

Conversation

@jacobperron
Copy link
Copy Markdown
Member

Backport #280 to Foxy.

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

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@jacobperron
Copy link
Copy Markdown
Member Author

I also had to backport a similar change for the tf2_ros Python package (4d60c02). This is a partial backport of the change on the ros2 branch during the split of CMake and Python packages in #210.

This is a backport of the change made when splitting out the Python module into its own packages in #210.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
@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
Copy link
Copy Markdown
Member Author

I thought ros-infrastructure/ros_buildfarm#828 would resolve the Fpr failure 🤔

Maybe it's failing because we've only changed the junit_family option for a subset of the packages being tested in this repo. I can try updating the option for all packages and see what happens.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette
Copy link
Copy Markdown
Contributor

Maybe it's failing because we've only changed the junit_family option for a subset of the packages being tested in this repo. I can try updating the option for all packages and see what happens.

Yeah, you need to have it for all of the packages. This is looking good to me.

@jacobperron jacobperron merged commit 12344e7 into foxy Aug 18, 2020
@jacobperron jacobperron deleted the jacob/backport_280 branch August 18, 2020 16:41
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