Skip to content

fix: pin external references to SHA for supply chain observability#848

Open
kitsuyui wants to merge 1 commit into
mainfrom
fix/supply-chain-renovate-preset-unpinned-001
Open

fix: pin external references to SHA for supply chain observability#848
kitsuyui wants to merge 1 commit into
mainfrom
fix/supply-chain-renovate-preset-unpinned-001

Conversation

@kitsuyui

Copy link
Copy Markdown
Owner

Summary

Pin two mutable external references to specific commit SHAs to improve supply chain observability.

Changes

.github/renovate.json5 — pin Renovate preset to SHA:

- "local>kitsuyui/renovate-config"
+ "local>kitsuyui/renovate-config#cdad3c71e34e5abadf909c996906011bbc2b5725"

.github/workflows/spellcheck.yml — pin reusable workflow to SHA:

- uses: kitsuyui/gh-actions-workflows/.github/workflows/spellcheck.yml@main
+ uses: kitsuyui/gh-actions-workflows/.github/workflows/spellcheck.yml@8019e658f4665c962d0a88eb38b3b6be418e7be8

Why

Without pinning, changes to kitsuyui/renovate-config and kitsuyui/gh-actions-workflows are silently reflected in this repo's Renovate behaviour and CI on every run — with no trace in this repo's commit history.

gh-counter.yml and gitignore-in.yml already use SHA-pinned references; this PR brings renovate.json5 and spellcheck.yml into alignment.

Verification

  • bun run lint: clean (biome check passes)
  • No functional changes to CI logic or Renovate rules

Pin two mutable external references to specific commit SHAs:

1. `.github/renovate.json5`: `local>kitsuyui/renovate-config` → pin to
   SHA cdad3c71. Without pinning, Renovate fetches the preset HEAD on
   every run, making behaviour changes untraceable in this repo's history.

2. `.github/workflows/spellcheck.yml`: `@main` → pin to SHA 8019e658.
   This aligns with the SHA-pinned convention already used in
   gh-counter.yml and gitignore-in.yml.

Both changes improve supply chain observability: preset/workflow updates
become explicit commits in this repo rather than silent side effects.
@github-actions

Copy link
Copy Markdown

🎉 Happy commit!

@github-actions

Copy link
Copy Markdown

Code Metrics Report

main (8bad195) #848 (f396866) +/-
Coverage 99.2% 99.2% 0.0%
Code to Test Ratio 1:1.3 1:1.3 0.0
Test Execution Time 2s 1s -1s
Details
  |                     | main (8bad195) | #848 (f396866) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          99.2% |          99.2% | 0.0% |
  |   Files             |              5 |              5 |    0 |
  |   Lines             |            134 |            134 |    0 |
  |   Covered           |            133 |            133 |    0 |
  | Code to Test Ratio  |          1:1.3 |          1:1.3 |  0.0 |
  |   Code              |            723 |            723 |    0 |
  |   Test              |            955 |            955 |    0 |
+ | Test Execution Time |             2s |             1s |  -1s |

Reported by octocov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant