This PR makes the IWYU output in the CI 'tidy' task more useful by avoiding most cases where a comment ends with an ellipsis like that:
#include "primitives/transaction.h" // for CTxIn, CMutableTransaction, CTra...
ci, iwyu: Double maximum line length for includes #27707
pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:230520-tidy changing 1 files +1 −0-
hebasto commented at 12:21 PM on May 20, 2023: member
-
ci, iwyu: Double maximum line length for includes 98ea798411
-
DrahtBot commented at 12:21 PM on May 20, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK TheCharlatan 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:
- #27710 (ci, iwyu: Update mappings 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.
- TheCharlatan approved
-
TheCharlatan commented at 7:33 PM on May 20, 2023: contributor
ACK 98ea79841177cf7492ec1de1ba1979f75390c63b
- maflcko approved
-
maflcko commented at 6:20 AM on May 22, 2023: member
lgtm
- fanquake merged this on May 22, 2023
- fanquake closed this on May 22, 2023
- hebasto deleted the branch on May 22, 2023
- sidhujag referenced this in commit 1bd30cb21f on May 23, 2023
- bitcoin locked this on May 21, 2024
Contributors