.
. #35190
pull fitzyracing1 wants to merge 1 commits into bitcoin:master from fitzyracing1:fix/rust-lint-expect-messages changing 3 files +12 −6-
fitzyracing1 commented at 1:17 AM on May 1, 2026: none
-
7b5ec7fddb
test: replace unwrap()/panic!() with expect() in lint test runner
Replace 6 instances of silent .unwrap() and panic!() calls with descriptive .expect() messages in the Rust lint test runner. This makes failures easier to diagnose by surfacing context in the panic message rather than a bare 'called Option::unwrap() on a None value'. Files changed: - test/lint/test_runner/src/lint_cpp.rs - test/lint/test_runner/src/util.rs - test/lint/test_runner/src/main.rs
- DrahtBot added the label Tests on May 1, 2026
-
DrahtBot commented at 1:17 AM on May 1, 2026: 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. A summary of reviews will appear here.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
sedited commented at 5:24 AM on May 1, 2026: contributor
AI slop
- sedited closed this on May 1, 2026
- fanquake renamed this:
test: replace unwrap()/panic!() with expect() in lint test runner
.
on May 1, 2026 - DrahtBot removed the label Tests on May 1, 2026
- bitcoin locked this on May 1, 2026
Contributors