Skip to content

feat(clang-tidy): Fix all warnings and errors for all C++ source files. #764

@davidlion

Description

@davidlion

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:

  1. Link to this issue for tracking.
  2. Remove the file from lint:check-cpp-static-full's exclude list.
  3. Comment on this issue, documenting any difficult fixes or discussion/guidelines that resulted from fixes.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions