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

[foxy backport] Backports for init_options test (#1165, #1272) #1388

Merged

Conversation

brawner
Copy link
Contributor

@brawner brawner commented Oct 6, 2020

This PR backports the init_options test from #1165 and #1272. It does not include the API changes and excludes the get/set_domain_id unit test.

@brawner brawner requested a review from ahcorde October 6, 2020 23:50
@brawner
Copy link
Contributor Author

brawner commented Oct 7, 2020

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

@brawner brawner force-pushed the brawner/rclcpp-backport-tests-part2 branch from f4ff384 to def9730 Compare October 7, 2020 18:30
Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>

Remove non-foxy api changes

Signed-off-by: Stephen Brawner <brawner@gmail.com>
Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@brawner brawner force-pushed the brawner/rclcpp-backport-1272 branch from be9139a to 4552f91 Compare October 7, 2020 18:35
@brawner brawner changed the base branch from brawner/rclcpp-backport-tests-part2 to brawner/rclcpp-backport-tests-part1 October 7, 2020 18:35
@brawner
Copy link
Contributor Author

brawner commented Oct 7, 2020

Rebase and merging onto #1383 so that PR can act as the ultimate feature branch.

@brawner brawner merged commit ae4bb5c into brawner/rclcpp-backport-tests-part1 Oct 7, 2020
@delete-merged-branch delete-merged-branch bot deleted the brawner/rclcpp-backport-1272 branch October 7, 2020 18:36
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.

None yet

3 participants