Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate clang-tidy on the CI #8148

Open
gilles-peskine-arm opened this issue Sep 1, 2023 · 0 comments
Open

Investigate clang-tidy on the CI #8148

gilles-peskine-arm opened this issue Sep 1, 2023 · 0 comments
Labels
component-test Test framework and CI scripts enhancement size-m Estimated task size: medium (~1w)

Comments

@gilles-peskine-arm
Copy link
Contributor

clang-tidy can run many sanity checks (linting) on our code base. This has come up before in the context of style checks, but there are also semantic checks that could be useful.

Some examples of potentially useful checks:

The goal of this task is to:

  • Either conclude that clang-tidy is too much cost for too little benefit, or add a run of clang-tidy on the CI. (TBD: in what configuration(s)) If we decide to run clang-tidy...
  • Define an initial set of checks.
  • Retire any ad hoc code that is made obsolete by the initial set of checks.
  • Figure out how to work around false positives.
  • File issues to investigate more checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component-test Test framework and CI scripts enhancement size-m Estimated task size: medium (~1w)
Projects
Status: No status
Development

No branches or pull requests

1 participant