As title says.
ci: Drop no longer needed `-I` flag in "tidy" task #29929
pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:240421-ci-tidy changing 1 files +1 −1-
hebasto commented at 1:41 PM on April 21, 2024: member
-
ci: Drop no longer needed `-I` flag in "tidy" task 6f5954acac
- hebasto added the label Tests on Apr 21, 2024
-
DrahtBot commented at 1:41 PM on April 21, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
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.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #29790 ([DO NOT MERGE] cmake: Migrate CI scripts to CMake-based build system -- WIP by hebasto)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
-
maflcko commented at 2:01 PM on April 21, 2024: member
lgtm ACK 6f5954acac2ced22dae7088e2b679bf663507d4c
- DrahtBot added the label CI failed on Apr 21, 2024
- DrahtBot removed the label CI failed on Apr 23, 2024
-
fanquake commented at 9:56 AM on April 24, 2024: member
Did this change with LLVM version, or IWYU version or changing the base image?
-
hebasto commented at 9:58 AM on April 24, 2024: member
Did this change with LLVM version, or IWYU version or changing the base image?
My guess is LLVM version, but I didn't check.
- fanquake merged this on Apr 24, 2024
- fanquake closed this on Apr 24, 2024
- hebasto deleted the branch on Apr 24, 2024
- bitcoin locked this on Apr 24, 2025