Skip to content

Update SHAs for actions in workflows#96

Merged
kevinb-khan merged 2 commits intomainfrom
update-shas
Mar 10, 2026
Merged

Update SHAs for actions in workflows#96
kevinb-khan merged 2 commits intomainfrom
update-shas

Conversation

@kevinb-khan
Copy link
Copy Markdown
Contributor

@kevinb-khan kevinb-khan commented Mar 9, 2026

Summary:

I've recently published Khan/actions so that the changes that @jaredly made in Khan/actions#128 are available now. This should unblock making updates to this repo. This PR updates all of the SHAs for the actions we're using in our workflows to make sure we're using actions that have all of their actions pinned. All of the actions in a dependency tree of actions is required.

Note

I had to update the versions in the comments to be very specific versions because the updated publish process in Khan/actions doesn't seem to be updating the major-only tags.

Issue: None

Test plan:

  • let CI run, if it passes then that will indicate that changes to Khan/actions have been successful

@kevinb-khan kevinb-khan self-assigned this Mar 9, 2026
@kevinb-khan kevinb-khan marked this pull request as draft March 9, 2026 19:15
@kevinb-khan kevinb-khan marked this pull request as ready for review March 9, 2026 19:27
@khan-actions-bot khan-actions-bot requested review from a team, jaredly and jeresig and removed request for a team March 9, 2026 19:28
@khan-actions-bot
Copy link
Copy Markdown
Contributor

Gerald

Required Reviewers
  • @Khan/github-actions for changes to .github/workflows/gerald-comment.yml, .github/workflows/gerald-pr.yml, .github/workflows/gerald-push.yml, .github/workflows/pr-autofix.yml

Don't want to be involved in this pull request? Comment #removeme and we won't notify you of further changes.

@kevinb-khan kevinb-khan requested review from a team, jeresig and somewhatabstract and removed request for a team, jaredly and jeresig March 9, 2026 19:28
Copy link
Copy Markdown
Member

@jeresig jeresig left a comment

Choose a reason for hiding this comment

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

Excellent - thank you!

@kevinb-khan kevinb-khan merged commit 70c7baf into main Mar 10, 2026
6 checks passed
@kevinb-khan kevinb-khan deleted the update-shas branch March 10, 2026 15:42
kevinb-khan added a commit that referenced this pull request Mar 11, 2026
## Summary:
In #96 I changed the versions tags in the comments to use pinned versions.  This was due to a change in Khan/actions that stopped the publish script from updating the major-only tags.  Khan/actions#146 has restored this behaviour which means that Khan/gerald can again use major-only versions in comments specifying versions of GitHub actions.

Issue: None

## Test plan:
- see that CI checks pass

Author: kevinb-khan

Reviewers: jeresig, somewhatabstract

Required Reviewers:

Approved By: jeresig

Checks: ✅ 4 checks were successful

Pull Request URL: #99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants