Skip to content

Make sure to include-what-you-use in the node_interfaces.#2018

Merged
clalancette merged 1 commit intorollingfrom
clalancette/rclcpp-node-interface-includes
Sep 29, 2022
Merged

Make sure to include-what-you-use in the node_interfaces.#2018
clalancette merged 1 commit intorollingfrom
clalancette/rclcpp-node-interface-includes

Conversation

@clalancette
Copy link
Copy Markdown
Contributor

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

This may help the compile error we are seeing in ros2/geometry2#555 . Even if it doesn't, it is the right thing to do.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette
Copy link
Copy Markdown
Contributor Author

CI:

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

@clalancette clalancette merged commit 95837d3 into rolling Sep 29, 2022
@delete-merged-branch delete-merged-branch bot deleted the clalancette/rclcpp-node-interface-includes branch September 29, 2022 11:54
alsora pushed a commit to irobot-ros/rclcpp that referenced this pull request Mar 24, 2023
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
alsora pushed a commit to irobot-ros/rclcpp that referenced this pull request Mar 24, 2023
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
alsora pushed a commit to irobot-ros/rclcpp that referenced this pull request Apr 29, 2023
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
alsora pushed a commit to irobot-ros/rclcpp that referenced this pull request Apr 29, 2023
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
alsora pushed a commit to irobot-ros/rclcpp that referenced this pull request May 3, 2023
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
mauropasse pushed a commit to mauropasse/rclcpp that referenced this pull request Jul 19, 2023
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Z572 added a commit to revyos-ros/rosdistro that referenced this pull request Jun 3, 2024
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