Skip to content

[foxy backport] Remove rmw-dependent unit-test checks (#1293)#1377

Merged
brawner merged 1 commit intofoxyfrom
brawner/rclcpp_lifecycle-backport-1293
Oct 5, 2020
Merged

[foxy backport] Remove rmw-dependent unit-test checks (#1293)#1377
brawner merged 1 commit intofoxyfrom
brawner/rclcpp_lifecycle-backport-1293

Conversation

@brawner
Copy link
Copy Markdown
Contributor

@brawner brawner commented Oct 5, 2020

  • Remove rmw-dependent unit-test checks

Signed-off-by: Stephen Brawner brawner@gmail.com

  • Address feedback

Signed-off-by: Stephen Brawner brawner@gmail.com

  • PR Fixup

Signed-off-by: Stephen Brawner brawner@gmail.com

* Remove rmw-dependent unit-test checks

Signed-off-by: Stephen Brawner <brawner@gmail.com>

* Address feedback

Signed-off-by: Stephen Brawner <brawner@gmail.com>

* PR Fixup

Signed-off-by: Stephen Brawner <brawner@gmail.com>
@brawner brawner requested a review from jacobperron October 5, 2020 18:36
@brawner
Copy link
Copy Markdown
Contributor Author

brawner commented Oct 5, 2020

Standard foxy build --packages-select rclcpp_lifecycle

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

@brawner brawner changed the title Remove rmw-dependent unit-test checks (#1293) [foxy backport] Remove rmw-dependent unit-test checks (#1293) Oct 5, 2020
@brawner
Copy link
Copy Markdown
Contributor Author

brawner commented Oct 5, 2020

cyclonedds only

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

@jacobperron
Copy link
Copy Markdown
Member

@ros-pull-request-builder retest this please

@brawner
Copy link
Copy Markdown
Contributor Author

brawner commented Oct 5, 2020

This is all green except for test_lifecycle_service_client on MacOS, which encountered a race condition unrelated to this change (since this PR just affects test_lifecycle_node.cpp)

@brawner brawner merged commit b843c75 into foxy Oct 5, 2020
@delete-merged-branch delete-merged-branch bot deleted the brawner/rclcpp_lifecycle-backport-1293 branch October 5, 2020 20:38
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