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

Removed sensor_msgs::msg::PointCloud references because is deprecated #1224

Open
wants to merge 3 commits into
base: rolling
Choose a base branch
from

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Jul 4, 2024

Related to this PR ros2/common_interfaces#254

@ahcorde ahcorde requested a review from clalancette July 4, 2024 15:41
@ahcorde ahcorde self-assigned this Jul 4, 2024
@ahcorde
Copy link
Contributor Author

ahcorde commented Jul 8, 2024

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Signed-off-by: Alejandro Hernández Cordero <[email protected]>
@ahcorde
Copy link
Contributor Author

ahcorde commented Jul 10, 2024

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@ahcorde
Copy link
Contributor Author

ahcorde commented Jul 10, 2024

@clalancette what do you think about this one ?

@ahcorde
Copy link
Contributor Author

ahcorde commented Jul 29, 2024

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor

@clalancette what do you think about this one ?

See my comment in ros2/common_interfaces#254 (comment) ; I'm not totally convinced we should do this right now.

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