-
Couldn't load subscription status.
- Fork 6.8k
Open
Labels
P2Important issue, but not time-criticalImportant issue, but not time-criticalci-testcommunity-backlogcoreIssues that should be addressed in Ray CoreIssues that should be addressed in Ray Coretech-debtThe issue that's due to tech debtThe issue that's due to tech debt
Description
Follow up #52275 . IWYU is enabled at the example folder bazel/cpp/test in ci pipeline now and we can also run bazel build --iwyu <target> to perform iwyu anaylsis.
Lines 30 to 36 in fe0c32a
| - label: ":coral: reef: iwyu tests" | |
| commands: | |
| - bazel test --config iwyu //bazel/tests/cpp:example_test | |
| instance_type: small | |
| depends_on: oss-ci-base_build | |
| job_env: oss-ci-base_build | |
| tags: tools |
It's time to cover IWYU for all C++ folders to reduce redundant headers and implicit inclusion.
Rough idea:
- IWYU CI pipeline for a small C++ folder
- Perform IWYU cleanup by folder and add to IWYU ci
dentiny
Metadata
Metadata
Assignees
Labels
P2Important issue, but not time-criticalImportant issue, but not time-criticalci-testcommunity-backlogcoreIssues that should be addressed in Ray CoreIssues that should be addressed in Ray Coretech-debtThe issue that's due to tech debtThe issue that's due to tech debt