Removes trailing whitespace that breaks the upstream Bitcoin Core build, and enables the warning on the ci here so it does not recur.
Remove trailing whitespace and Add -Wtrailing-whitespace to default ci config #345
pull ViniciusCestarii wants to merge 2 commits into bitcoin-core:master from ViniciusCestarii:remove-trailing-whitespace changing 2 files +2 −2-
ViniciusCestarii commented at 8:22 PM on August 14, 2026: contributor
-
refactor: Remove trailing whitespace 2f4be9ec6e
-
ci: Add -Wtrailing-whitespace to default config 9b136782af
-
DrahtBot commented at 8:22 PM on August 14, 2026: none
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline and AI policy for information on the review process.
Type Reviewers ACK ryanofsky If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
ryanofsky commented at 10:10 PM on August 14, 2026: collaborator
Code review ACK 9b136782af109206a7beeb73e23d99486f25400e
- It might be a good idea to keep default.bash and llvm.bash flags in sync if both compilers support this.
- Could also fix the missing newline at the end of type-unordered-set.h which the linter flags (https://github.com/bitcoin/bitcoin/pull/32387/changes/f409893f9b65244f03cff13d9de963165f762fe1)
- ryanofsky merged this on Aug 14, 2026
- ryanofsky closed this on Aug 14, 2026
Contributors