ci(core): reduce coverage threshold for T3B1
What the developer wrote
Authored by Roman Zeyde
ci(core): reduce coverage threshold for T3B1
[no changelog]
What changed, and why it matters
This commit only changes a CI (continuous integration) test coverage threshold for one hardware model (T3B1) from 85% to 84%, with a TODO comment linking to an internal issue. It does not modify any firmware code, cryptography, or device behavior. There is no security-relevant change to the product itself.
No security action required. Treat as routine CI maintenance. Optionally monitor issue #5957 to ensure the coverage drop is temporary and understood.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch updates .github/workflows/core.yml to lower the COVERAGE_THRESHOLD for the T3B1 model from 85 to 84 using a conditional GitHub Actions expression. The change is accompanied by a TODO referencing issue #5957. No source code, build scripts, or device firmware logic is altered.
Changed components
.github/workflows/core.ymlInspect captured patch +2 / −1
diff --git a/.github/workflows/core.yml b/.github/workflows/core.yml
index dd3dd3299..73fa4bb3c 100644
--- a/.github/workflows/core.yml
+++ b/.github/workflows/core.yml
@@ -651,7 +651,8 @@ jobs:
matrix:
model: [T2T1, T3B1, T3T1, T3W1]
env:
- COVERAGE_THRESHOLD: 85
+ # TODO: https://github.com/trezor/trezor-firmware/issues/5957
+ COVERAGE_THRESHOLD: ${{ matrix.model == 'T3B1' && 84 || 85 }}
steps:
- uses: actions/checkout@v4
with:
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.