Skip to content

Convert guard condition functions to pybind11#692

Merged
sloretz merged 1 commit intomasterfrom
pybind11_guard_condition
Mar 6, 2021
Merged

Convert guard condition functions to pybind11#692
sloretz merged 1 commit intomasterfrom
pybind11_guard_condition

Conversation

@sloretz
Copy link
Copy Markdown
Contributor

@sloretz sloretz commented Mar 3, 2021

Part of #665

This converts a couple guard condition functions to pybind11.

  • rclpy_create_guard_condition
  • rclpy_trigger_guard_condition

@sloretz sloretz self-assigned this Mar 3, 2021
@sloretz sloretz mentioned this pull request Mar 3, 2021
34 tasks
@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Mar 3, 2021

CI (build: --packages-up-to rclpy test: --packages-select rclpy)

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status
    • Windows re-run to see if chef cookbook issue is temporary Build Status

rclpy_create_guard_condition
rclpy_trigger_guard_condition

Signed-off-by: Shane Loretz <sloretz@openrobotics.org>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz force-pushed the pybind11_guard_condition branch from 437e4e8 to 254103e Compare March 6, 2021 00:05
@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Mar 6, 2021

Rebased - no changes. Running full CI since windows had an issue.

CI (build: --packages-up-to rclpy test: --packages-select rclpy)

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

@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Mar 6, 2021

Merging since CI is green and the rebase fixed conflicts without changing any logic.

@sloretz sloretz merged commit 9d75518 into master Mar 6, 2021
@delete-merged-branch delete-merged-branch bot deleted the pybind11_guard_condition branch March 6, 2021 00:59
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