ci: Use YAML anchor and aliases for repeated "Print logs" steps
What changed, and why it matters
This commit is a routine cleanup of the project's automated testing configuration file. It replaces several copy-pasted 'Print logs' steps with a single reusable template reference, making the file shorter and easier to maintain. There is no change to the actual software code, no change to how tests run, and no security relevance.
No action required. This is a non-security CI refactoring.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies .github/workflows/ci.yml to introduce a YAML anchor (&PRINT_LOGS) and reuse it via aliases (*PRINT_LOGS) for repeated ‘Print logs’ GitHub Actions steps. The behavior of the CI workflow is unchanged; only duplication is removed. No cryptographic, build, or runtime code is affected.
Changed components
.github/workflows/ci.ymlInspect captured patch +14 / −41
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 6f7b51d..7195b7d 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -117,7 +117,8 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
+ - &PRINT_LOGS
+ name: Print logs
uses: ./.github/actions/print-logs
if: ${{ !cancelled() }}
@@ -151,9 +152,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
s390x_debian:
name: "s390x (big-endian): Linux (Debian stable, QEMU)"
@@ -181,10 +180,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
-
+ - *PRINT_LOGS
arm32_debian:
name: "ARM32: Linux (Debian stable, QEMU)"
@@ -220,9 +216,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
arm64-debian:
name: "arm64: Linux (Debian stable)"
@@ -258,9 +252,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
ppc64le_debian:
name: "ppc64le: Linux (Debian stable, QEMU)"
@@ -288,10 +280,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
-
+ - *PRINT_LOGS
valgrind_debian:
name: "Valgrind ${{ matrix.binary_arch }} (memcheck)"
@@ -343,9 +332,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
sanitizers_debian:
name: "UBSan, ASan, LSan"
@@ -385,9 +372,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
msan_debian:
name: "MSan"
@@ -434,10 +419,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
-
+ - *PRINT_LOGS
mingw_debian:
name: ${{ matrix.configuration.job_name }}
@@ -475,9 +457,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
x86_64-macos-native:
name: "x86_64: macOS Ventura, Valgrind"
@@ -526,9 +506,7 @@ jobs:
python3 -m pip install lief
python3 ./tools/symbol-check.py .libs/libsecp256k1.dylib
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
arm64-macos-native:
name: "ARM64: macOS Sonoma"
@@ -578,10 +556,7 @@ jobs:
python3 -m pip install lief
python3 ./tools/symbol-check.py .libs/libsecp256k1.dylib
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
-
+ - *PRINT_LOGS
win64-native:
name: ${{ matrix.configuration.job_name }}
@@ -685,9 +660,7 @@ jobs:
with:
dockerfile: ./ci/linux-debian.Dockerfile
- - name: Print logs
- uses: ./.github/actions/print-logs
- if: ${{ !cancelled() }}
+ - *PRINT_LOGS
cxx_headers_debian:
name: "C++ (public headers)"
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.