doc: Remove build instruction for running clang-tidy #32662

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:250602-doc-clang-tidy changing 1 files +0 −1
  1. hebasto commented at 12:50 pm on June 2, 2025: member
    One of the benefits of using a compilation database, which is available after the CMake build system generation step, is that it is not necessary to actually build the code in order to run clang-tidy.
  2. doc: Remove build instruction for running `clang-tidy`
    One of the benefits of using a compilation database, which is available
    after the CMake build system generation step, is that it is not
    necessary to actually build the code in order to run `clang-tidy`.
    4b1b36acb4
  3. hebasto added the label Docs on Jun 2, 2025
  4. DrahtBot commented at 12:50 pm on June 2, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32662.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK TheCharlatan, janb84

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    LLM Linter (✨ experimental)

    Possible typos and grammar issues:

    • “denoised of errors” → “denoised, with errors removed” [“denoised of” is not idiomatic English]

    drahtbot_id_4_m

  5. TheCharlatan approved
  6. TheCharlatan commented at 1:43 pm on June 2, 2025: contributor
    ACK 4b1b36acb48fab133ca4a3241148fa9683915874
  7. janb84 commented at 1:44 pm on June 2, 2025: contributor

    ACK 4b1b36acb48fab133ca4a3241148fa9683915874

    Checked both outcomes, the “old” way and the new way, both outcomes are the equal.(the diff was harder to check due to running it in multithreaded mode, therefor the files where out of order)

  8. fanquake merged this on Jun 2, 2025
  9. fanquake closed this on Jun 2, 2025

  10. hebasto deleted the branch on Jun 2, 2025
  11. maflcko commented at 1:37 pm on June 3, 2025: member

    Shouldn’t this be excluded in the tidy CI task as well?

    Also, would be good to mention that this will now print errors about the missing auto-generated files.


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: 2025-06-15 06:13 UTC

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