ci: Fix filtering out Qt-generated files from compile_commands.json #31928

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:250221-autogen changing 1 files +6 −4
  1. hebasto commented at 1:32 pm on February 21, 2025: member

    This PR:

    1. Adjusts the regex for Qt-generated files to match the CMake build directory structure.
    2. Moves the filtering command to run before clang-tidy, ensuring that Qt-generated files are not needlessly processed.

    Fixes #31801.

  2. ci: Fix filtering out Qt generated files from `compile_commands.json` d82dc10415
  3. hebasto added the label Tests on Feb 21, 2025
  4. DrahtBot commented at 1:32 pm on February 21, 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/31928.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK maflcko

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

  5. maflcko commented at 2:23 pm on February 21, 2025: member

    ACK d82dc1041524e8402d201d32c9143233b5ec1baf 🚂

    Signature:

    0untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
    1RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
    2trusted comment: ACK d82dc1041524e8402d201d32c9143233b5ec1baf 🚂
    3VxeXCGLGGD5Mokc9xIUZQZRwrg91hsJUs6+zbThTAgoIarCrQx8s0lJ9WQaCwte6blo2y+lyqFbwQZEKXjVKAw==
    
  6. fanquake merged this on Feb 21, 2025
  7. fanquake closed this on Feb 21, 2025

  8. hebasto deleted the branch on Feb 21, 2025
  9. pablomartin4btc approved
  10. pablomartin4btc commented at 4:42 pm on February 21, 2025: member

    post-merge ACK d82dc1041524e8402d201d32c9143233b5ec1baf

    Verified that the regex adjustment successfully excludes CMake-generated files (bitcoinqt_autogen/), as evidenced by the absence of clang-tidy runs on these files in the updated logs.


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-02-22 06:12 UTC

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