Skip to content

feat(cli): pin GitHub Action version default to the released version#713

Merged
Chemaclass merged 3 commits into
mainfrom
feat/pin-action-version
Jun 9, 2026
Merged

feat(cli): pin GitHub Action version default to the released version#713
Chemaclass merged 3 commits into
mainfrom
feat/pin-action-version

Conversation

@Chemaclass

Copy link
Copy Markdown
Member

What

The action's version input defaulted to latest, which reads like a rolling tag even though install.sh already ships a baked, release-bumped LATEST_BASHUNIT_VERSION and never makes a runtime "newest release" lookup. So pinning the action by SHA already pinned the version — the default just hid that.

This makes the default a concrete version string in action.yml, bumped by the release process, so the pinned version is visible right at the call site. Pinning the action by SHA pins a visible bashunit version with no ambiguity.

Changes

  • action.yml: version default latest'0.38.0'; clearer description
  • release.sh: new release::update_action_version, added action.yml to RELEASE_FILES, wired into both the sandbox and real release steps (with state recording)
  • regression test + mock_action.yml fixture: bumps only the numeric default, leaves lib/'true' untouched
  • CHANGELOG.md entry

Notes

No behavior change for callers passing version: latest explicitly — that still installs the build-time pinned tag. Context: phpstan-src#5826 review.

Checks

  • make sa, make lint pass
  • ./bashunit tests/ and ./bashunit --parallel tests/ green

The action's 'version' input defaulted to 'latest', which read as a
rolling tag even though install.sh already ships a baked, release-bumped
version. Default it to the concrete version instead, bumped by the
release process, so pinning the action by SHA pins a visible version at
the call site. Pass 'version: latest' to keep the build-time tag.
@Chemaclass Chemaclass added the enhancement New feature or request label Jun 9, 2026
@Chemaclass Chemaclass self-assigned this Jun 9, 2026
@Chemaclass Chemaclass merged commit c8463bf into main Jun 9, 2026
34 checks passed
@Chemaclass Chemaclass deleted the feat/pin-action-version branch June 9, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants