Skip to content

Revert "Force xunit2 for pytest 6.0 when legacy mode is enabled"#468

Merged
nuclearsandwich merged 1 commit intomasterfrom
revert-415-pytest6_xunit
Jun 8, 2020
Merged

Revert "Force xunit2 for pytest 6.0 when legacy mode is enabled"#468
nuclearsandwich merged 1 commit intomasterfrom
revert-415-pytest6_xunit

Conversation

@nuclearsandwich
Copy link
Copy Markdown
Member

Reverts #415 in order to restore Windows and macOS CI which is broken by the absence of pytest in early stages of the build on those platforms. #467 is open to track restoring #415 with corrected behavior after this revert is merged.

@nuclearsandwich nuclearsandwich self-assigned this Jun 8, 2020
@nuclearsandwich nuclearsandwich requested a review from j-rivero June 8, 2020 13:36
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.

Looks good to me with green CI (particularly on Windows and macOS).

@nuclearsandwich
Copy link
Copy Markdown
Member Author

nuclearsandwich commented Jun 8, 2020

CI up to rcutils only since we just want to know that builds can run.

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

Extra macos build on mini1 which was the problem node in the nightlies Build Status

@nuclearsandwich nuclearsandwich merged commit b286bd7 into master Jun 8, 2020
@nuclearsandwich nuclearsandwich deleted the revert-415-pytest6_xunit branch June 8, 2020 13:47
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