Skip to content

docs: add CONTRIBUTING.md with anchor checksum update process#52

Open
kitsuyui wants to merge 1 commit into
mainfrom
fix/audit-anchor-checksum-update-process-undocumented-001
Open

docs: add CONTRIBUTING.md with anchor checksum update process#52
kitsuyui wants to merge 1 commit into
mainfrom
fix/audit-anchor-checksum-update-process-undocumented-001

Conversation

@kitsuyui

Copy link
Copy Markdown
Member

Why

When Renovate (or a contributor) bumps the pinned gibo version in action.yml,
two SHA256 anchor values — checksums_txt_sha256 and checksums_windows_txt_sha256
must also be updated to match the new release. Renovate's custom manager only updates
the version line; it cannot update the anchor hashes. Leaving them stale causes the
action's anchor verification step to fail on the first CI run after a version bump.

No written procedure for this update existed in the repo: CONTRIBUTING.md was absent,
and README.md's Security model section explained the why but not the how.

What changed

  • Added CONTRIBUTING.md with a step-by-step guide for bumping the gibo version:
    • Download the new checksums.txt and checksums.windows.txt from the new release.
    • Compute their SHA256 hashes with sha256sum (or shasum -a 256 on macOS).
    • Update all three values in action.yml (version, checksums_txt_sha256, checksums_windows_txt_sha256).
    • Verify the update locally before pushing.

Verification

  • typos CONTRIBUTING.md — no spelling errors.
  • Collateral check (check-pr-collateral.py) — clean, no violations.
  • No action.yml changes; the Example workflow CI is unaffected by a documentation-only commit.

Documents the manual steps required when bumping the pinned gibo version:
download the new checksums.txt and checksums.windows.txt from the upstream
release, compute their SHA256 hashes, and update the two anchor values in
action.yml alongside the version change.

Without this documentation, contributors who receive a Renovate PR that
updates only the version line have no written procedure for the required
anchor update, causing CI to fail with no clear explanation.
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