ci: Use clang-15 in "tidy" task#27311
Conversation
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. |
|
clang-16 was released recently, any reason not to switch to that directly while we're at it? |
|
lgtm ACK 8fe27fb
Sounds good to do this, once iwyu has a release with clang-16 support. Maybe in a follow-up? |
Well, it is available as a package in Ubuntu Lunar, but I'm not sure if we want to bring a short-lived distro into our CI. |
a56c965 ci: use clang-16 in tidy task (fanquake) Pull request description: Follow up to bitcoin/bitcoin#27311 (comment), as IWYU now has a [clang_16 branch](https://github.com/include-what-you-use/include-what-you-use/tree/clang_16). This also removes some workarounds for (now fixed) clang-tidy issues, and simplifies the IWYU install steps. ACKs for top commit: MarcoFalke: lgtm ACK a56c965 josibake: ACK bitcoin/bitcoin@a56c965 hebasto: ACK a56c965 Tree-SHA512: 5bbec6cc196c3305302895c77986f3695fc6f4024363ee57503654d54e0ebf108719a7a1d7908817f84115dcaa13377493eb764b00bdf574f1290c73251426fa
5 participants
Newer tools usually are better in terms of features and bug fixes.
Requested in #26642 (comment).
Split from #26766.