Skip to content

Commit 9a8318f

Browse files
committed
Merge bitcoin/bitcoin#27707: ci, iwyu: Double maximum line length for includes
98ea798 ci, iwyu: Double maximum line length for includes (Hennadii Stepanov) Pull request description: 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... ``` ACKs for top commit: TheCharlatan: ACK 98ea798 Tree-SHA512: 25195ccb2095884b23586416b86999ebc42577c6d777abdbd176a704fa75c64deb91fa61cd91d570a5408dd459e930e53bc70d963b76c73fca7a800e74b1bdbf
2 parents 17acb27 + 98ea798 commit 9a8318f

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

ci/test/06_script_b.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ if [ "${RUN_TIDY}" = "true" ]; then
166166
python3 "${DIR_IWYU}/include-what-you-use/iwyu_tool.py" \
167167
-p . "${MAKEJOBS}" \
168168
-- -Xiwyu --cxx17ns -Xiwyu --mapping_file="${BASE_BUILD_DIR}/bitcoin-$HOST/contrib/devtools/iwyu/bitcoin.core.imp" \
169+
-Xiwyu --max_line_length=160 \
169170
2>&1 | tee /tmp/iwyu_ci.out
170171
cd "${BASE_ROOT_DIR}/src"
171172
python3 "${DIR_IWYU}/include-what-you-use/fix_includes.py" --nosafe_headers < /tmp/iwyu_ci.out

0 commit comments

Comments
 (0)