Request
36140bf adds support for running clang-tidy through task and in the GH workflows.
Currently, the task lint:check-cpp-static-full succeeds thanks to an exclude list that skips all files with known clang-tidy issues (see https://github.com/y-scope/clp/blob/main/lint-tasks.yml for the task and exclude list).
The goal is to fix all the clang-tidy warnings and errors so that lint:check-cpp-static-full can succeed without the need for an exclude list.
Possible implementation
When you create a PR fixing clang-tidy issues:
- Link to this issue for tracking.
- Remove the file from
lint:check-cpp-static-full's exclude list.
- Comment on this issue, documenting any difficult fixes or discussion/guidelines that resulted from fixes.
Request
36140bf adds support for running clang-tidy through
taskand in the GH workflows.Currently, the task
lint:check-cpp-static-fullsucceeds thanks to an exclude list that skips all files with known clang-tidy issues (see https://github.com/y-scope/clp/blob/main/lint-tasks.yml for the task and exclude list).The goal is to fix all the clang-tidy warnings and errors so that
lint:check-cpp-static-fullcan succeed without the need for an exclude list.Possible implementation
When you create a PR fixing clang-tidy issues:
lint:check-cpp-static-full's exclude list.