ci: Switch IWYU to `clang_17` branch #28826

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:231108-iwyu changing 1 files +1 −2

hebasto commented at 7:15 PM on November 8, 2023: member

The IWYU version 0.21 has been tagged, and the clang_17 branch is available now.
DrahtBot commented at 7:15 PM on November 8, 2023: contributor



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



Code Coverage

For detailed information about the code coverage, see the test coverage report.



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.
DrahtBot added the label Tests on Nov 8, 2023

Type	Reviewers
ACK	maflcko

in ci/test/01_base_install.sh:70 in 26a2cc5e8a outdated

  66 | @@ -67,8 +67,7 @@ if [[ ${USE_MEMORY_SANITIZER} == "true" ]]; then
  67 |  fi
  68 |  
  69 |  if [[ "${RUN_TIDY}" == "true" ]]; then
  70 | -  ${CI_RETRY_EXE} git clone https://github.com/include-what-you-use/include-what-you-use -b master /include-what-you-use
  71 | -  git -C /include-what-you-use checkout a138eaac254e5a472464e31d5ec418fe6e6f1fc7
  72 | +  ${CI_RETRY_EXE} git clone --depth=1 https://github.com/include-what-you-use/include-what-you-use -b clang_17 /include-what-you-use

maflcko commented at 11:54 AM on November 9, 2023:

  ${CI_RETRY_EXE} git clone --depth=1 https://github.com/include-what-you-use/include-what-you-use -b clang_$TIDY_LLVM_V /include-what-you-use

hebasto commented at 12:05 PM on November 9, 2023:

Thanks! Done.

maflcko approved
ci: Switch IWYU to `clang_17` branch 9f208c0171
hebasto force-pushed on Nov 9, 2023
maflcko commented at 12:07 PM on November 9, 2023: member

lgtm ACK 9f208c017174132dbefbc917aa9824c279382597
fanquake merged this on Nov 9, 2023
fanquake closed this on Nov 9, 2023
hebasto deleted the branch on Nov 9, 2023
bitcoin locked this on Nov 12, 2024

Contributors

Labels

ci: Switch IWYU to `clang_17` branch #28826

Code Coverage

Reviews