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

issue fanquake openend this issue on April 3, 2022
  1. fanquake commented at 12:16 pm on April 3, 2022: member

    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.

  2. fanquake added the label good first issue on Apr 3, 2022
  3. MarcoFalke added the label Tests on Apr 4, 2022
  4. MarcoFalke commented at 6:54 am on April 4, 2022: member
    I think this requires a “mock” compilation so this will probably be a normal task, separate from the lint task.
  5. fanquake closed this on Apr 6, 2022

  6. DrahtBot locked this on Apr 6, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-09-28 22:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me