Skip to content

Commit 4373855

Browse files
nbbrooksclaude
andauthored
Remove CLANG_TIDY_ARGS to support run-clang-tidy (backport of #3640) (#3775)
Same fix as #3640 (main) and the equivalent humble/jazzy backports. The run-clang-tidy script shipped in current kilted-ci Docker images no longer accepts the `--fix --fix-errors --format-style=file` flags, so any kilted PR that touches a file clang-tidy actually checks (like PR #3774 touching moveit_setup_controllers) fails with: run-clang-tidy: error: unrecognized arguments: --fix --fix-errors --format-style=file 'clang_tidy_check_moveit_setup_controllers' returned with code '123' Drop the env var so run-clang-tidy gets invoked with its defaults. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 121e59d commit 4373855

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

.github/workflows/ci.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ jobs:
2929
# https://stackoverflow.com/a/41673702
3030
CXXFLAGS: >-
3131
-Wall -Wextra -Wwrite-strings -Wunreachable-code -Wpointer-arith -Wredundant-decls -Wno-unknown-warning-option
32-
CLANG_TIDY_ARGS: --fix --fix-errors --format-style=file
3332
DOCKER_IMAGE: moveit/moveit2:${{ matrix.env.IMAGE }}
3433
UPSTREAM_WORKSPACE: >
3534
moveit2.repos

0 commit comments

Comments
 (0)