chore(ci): update core-hw actions [no changelog]
What changed, and why it matters
This commit is a routine maintenance update to the project's automated testing configuration. It upgrades the versions of some GitHub Actions used in hardware test workflows and simplifies the checkout logic. There is no indication this changes the actual Trezor firmware code or introduces a security vulnerability.
No security action required. Review the updated GitHub Actions versions for compatibility with existing workflow behavior, as with any dependency update.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff updates .github/workflows/core-hw.yml. It bumps actions/checkout from commit 34e1148 (v4.3.1) to de0fac2 (v6.0.2) and actions/upload-artifact from ea165f8 (v4.6.2) to bbbca2d (v7.0.0). It also removes a conditional split that used different checkout versions for T2B1/T2T1 models versus others, and removes a model-specific condition on artifact upload. These are CI-only changes with no modifications to firmware source, build scripts, or cryptographic logic.
Changed components
.github/workflows/core-hw.ymlInspect captured patch +3 / −9
diff --git a/.github/workflows/core-hw.yml b/.github/workflows/core-hw.yml
index 38f343a9..897af98a 100644
--- a/.github/workflows/core-hw.yml
+++ b/.github/workflows/core-hw.yml
@@ -83,16 +83,10 @@ jobs:
TT_UHUB_PORT: all
timeout-minutes: 360 # 6h CI job timeout
steps:
- - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
- with:
- submodules: recursive
- persist-credentials: false
- if: matrix.model == 'T2B1' || matrix.model == 'T2T1'
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # actions/checkout@v6.0.2
with:
submodules: recursive
persist-credentials: false
- if: matrix.model != 'T2B1' && matrix.model != 'T2T1'
- uses: ./.github/actions/environment
- run: nix-shell --arg hardwareTest true --run uhubctl
- run: nix-shell --run "uv run make -C core build_firmware"
@@ -113,7 +107,7 @@ jobs:
trezor.log
pytest.log
retention-days: 7
- if: always() && matrix.model != 'T2B1' && matrix.model != 'T2T1'
+ if: always()
core_monero_test:
name: Monero tests
@@ -181,7 +175,7 @@ jobs:
BITCOIN_ONLY: ${{ matrix.coins == 'universal' && '0' || '1' }}
DEBUG_LINK: 1
steps:
- - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
+ - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # actions/checkout@v6.0.2
with:
submodules: recursive
persist-credentials: false
@@ -190,7 +184,7 @@ jobs:
- run: nix-shell --run "uv run legacy/script/setup"
- run: nix-shell --run "export PRODUCTION=0 && uv run legacy/script/cibuild"
- run: nix-shell --arg hardwareTest true --run "ci/hardware_tests/t1_hw_test.sh"
- - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # actions/upload-artifact@v4.6.2
+ - uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # actions/upload-artifact@v7.0.0
with:
name: legacy-hardware-${{ matrix.coins }}
path: ci/hardware_tests/*.mp4
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.