Skip to content

Include what you use - #592

Open
ahcorde wants to merge 1 commit into
rollingfrom
ahcorde/rolling/iwyu
Open

ahcorde wants to merge 1 commit into
rollingfrom
ahcorde/rolling/iwyu

Conversation

@ahcorde

@ahcorde ahcorde commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Description

Include what you use

Did you use Generative AI?

Claude Opus 4.8

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde self-assigned this Sep 9, 2026
@ahcorde

ahcorde commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Pulls: #592
Gist: https://gist.githubusercontent.com/ahcorde/1a48fa05cf1206efa60d659322fb0661/raw/8bb0162739ee67e1029cad1617464e5aa2b0e2e5/ros2.repos
BUILD args: --packages-above-and-dependencies rcutils
TEST args: --packages-above rcutils
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/20400

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

@fujitatomoya fujitatomoya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ahcorde i think dependent packages break the build because of the chain of the header files?

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.

3 participants