Skip to content

ci: add lint job to run clang-tidy check for named args #24747

@fanquake

Description

@fanquake

After #24661, we should be able to enable clang-tidy named argument checking in the CI lint job. The simplest way to invoke this checking, is something like ./autogen.sh && ./configure CC=clang-12 CXX=clang++-12 && make clean && bear make -j9 && ( cd ./src/ && run-clang-tidy-12 -j9 ), which invokes clang-tidy using our .clang-tidy config.

Dependencies, like clang-tidy would need to be installed in the linting container, and a linting job could be added to https://github.com/bitcoin/bitcoin/tree/master/test/lint.

It is possible to work on this now, by just basing your changes on the #24661 branch.

Useful skills:

  • Bash
  • CI

Want to work on this issue?

For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions