Skip to content

COMP: Pin mutable CI action references to fixed versions#81

Merged
hjmjohnson merged 1 commit into
InsightSoftwareConsortium:mainfrom
hjmjohnson:comp/ci-pin-action-refs
Jun 8, 2026
Merged

COMP: Pin mutable CI action references to fixed versions#81
hjmjohnson merged 1 commit into
InsightSoftwareConsortium:mainfrom
hjmjohnson:comp/ci-pin-action-refs

Conversation

@hjmjohnson

Copy link
Copy Markdown
Member

Pin the remaining floating/mutable GitHub Actions references to fixed versions so CI is reproducible and not silently affected by upstream action changes. Pure CI hygiene — no source or build-logic changes.

The highest-value fix is pypa/gh-action-pypi-publish@master@v1.14.0: that action runs the credentialed PyPI publish (user: __token__), so a moving branch ref is the riskiest reference in the tree.

All changes (6 refs across 4 workflows)
File Before After
build-test-package.yml pypa/gh-action-pypi-publish@master @v1.14.0
build-test-package.yml lukka/get-cmake@latest (×4) @v4.3.3
build-test-package.yml Jimver/cuda-toolkit@v0.2.21 @v0.2.35
test-gpu.yml lukka/get-cmake@v3.22.2 @v4.3.3
clang-format-linter.yml ITKClangFormatLinterAction@main @bab3a0bb26af8a23c7cc2c1d4265834415089eb5 (SHA; action has no release tags)
test-notebooks.yml actions/setup-python@v3 @v5
  • get-cmake@v4.3.3 is the release @latest currently resolves to, so the four @latest swaps are behavior-preserving.
  • get-cmake@v3.22.2 in test-gpu.yml bundles an @actions/cache that calls the retired Actions cache API and aborts fatally; dormant only because that job is gated. Unified to @v4.3.3.
  • All pin targets verified as published releases.
Validation

Workflow files can only be exercised by CI itself (no local run). All four YAML files parse, no mutable/stale refs remain, and pre-commit run --all-files passes on the branch.

Replace floating/mutable GitHub Actions refs with fixed versions so CI
is reproducible and not silently affected by upstream action changes:

- pypa/gh-action-pypi-publish: @master -> @v1.14.0. This action
  performs the credentialed PyPI publish (user: __token__), so a moving
  branch ref is the highest-risk floating reference in the tree.
- lukka/get-cmake: @latest (x4) -> @v4.3.3, the release @latest
  currently resolves to (no behavior change, just frozen).
- lukka/get-cmake in test-gpu.yml: @v3.22.2 -> @v4.3.3. v3.22.2 bundles
  an @actions/cache that calls the retired Actions cache API and aborts
  fatally; it is dormant only because that job is gated.
- InsightSoftwareConsortium/ITKClangFormatLinterAction: @main -> pinned
  commit SHA (the action publishes no release tags).
- Jimver/cuda-toolkit: @v0.2.21 -> @v0.2.35.
- actions/setup-python in test-notebooks.yml: @V3 -> @v5 to match the
  other jobs.
@hjmjohnson hjmjohnson marked this pull request as ready for review June 8, 2026 18:26
@hjmjohnson

Copy link
Copy Markdown
Member Author

@dzenanz Thanks! I hope this is the last PR here for a while. I am going to work on ITK remote module update of VkFFTBackend, and then the open ANTs PR.

@hjmjohnson hjmjohnson merged commit abcc6b5 into InsightSoftwareConsortium:main Jun 8, 2026
14 checks passed
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.

2 participants