Skip to content

Pin actions to SHAs - #230

Merged
keiravillekode merged 1 commit into
exercism:mainfrom
keiravillekode:pin-actions-cache
Jul 13, 2026
Merged

Pin actions to SHAs#230
keiravillekode merged 1 commit into
exercism:mainfrom
keiravillekode:pin-actions-cache

Conversation

@keiravillekode

@keiravillekode keiravillekode commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Update actions/cache to v6.1.0

We pin by SHA

  • actions/setup-python
  • actions/cache
  • pre-commit/action

https://exercism.org/docs/building/github/gha-best-practices#h-pin-actions-to-shas

@ahans ahans left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only saw this now, sorry for the wait!

Comment thread .github/workflows/pre-commit.yml Outdated
- name: set PY
run: echo "PY=$(python -VV | sha256sum | cut -d' ' -f1)" >> $GITHUB_ENV
- uses: actions/cache@v5
- uses: actions/cache@55cc8345863c7cc4c66a329aec7e433d2d1c52a9

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an upgrade to v6. This is fine, but goes pretty much undocumented now. Do you think it'd make sense to add a comment with the corresponding tag?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Update actions/cache to v6.1.0

We pin by SHA
- actions/setup-python
- actions/cache
- pre-commit/action

https://exercism.org/docs/building/github/gha-best-practices#h-pin-actions-to-shas
@keiravillekode
keiravillekode merged commit b466001 into exercism:main Jul 13, 2026
4 checks passed
@keiravillekode
keiravillekode deleted the pin-actions-cache branch July 13, 2026 09:57
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