What the developer wrote
Authored by Tobin C. Harding
units: Remove TODO comment
There is a fix in flight for this: #5875
In case we don't want to wait for that PR to land just remove the
comment so the pre-release job on #5891 can pass.
What changed, and why it matters
This commit simply removes a two-line TODO comment from a test file. No code behavior changes, no security fix or vulnerability is introduced.
No security action needed. Treat as routine cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff deletes a comment ‘// TODO: Implement Neg all round.’ in units/src/amount/tests.rs. The actual test code and all other logic remain unchanged. The commit message states the removal is to unblock a pre-release CI job while a real fix is being handled in PR #5875.
Changed components
units/src/amount/tests.rsInspect captured patch +0 / −2
diff --git a/units/src/amount/tests.rs b/units/src/amount/tests.rs
index ef1d0ce4..b613fe85 100644
--- a/units/src/amount/tests.rs
+++ b/units/src/amount/tests.rs
@@ -1391,8 +1391,6 @@ fn num_op_result_ops_integer() {
// Verify we have implemented all `Neg` for the amount types.
#[test]
fn amount_op_result_neg() {
- // TODO: Implement Neg all round.
-
// let sat = Amount::from_sat(1).unwrap();
let ssat = SignedAmount::from_sat(1).unwrap();
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.