Skip to content

chore(ci): pin all GHA#419

Merged
chris-olszewski merged 1 commit intomainfrom
olszewski/pin_gha
Apr 16, 2026
Merged

chore(ci): pin all GHA#419
chris-olszewski merged 1 commit intomainfrom
olszewski/pin_gha

Conversation

@chris-olszewski
Copy link
Copy Markdown
Member

What was changed

TSIA

Why?

Harden against a compromised GHA

Checklist

  1. Closes N/A

  2. How was this tested:
    CI

  3. Any docs updates needed?
    N/A

@chris-olszewski chris-olszewski requested a review from a team as a code owner April 16, 2026 18:00
steps:
- name: Checkout repository
uses: actions/checkout@v4
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What is the process for updating / maintaining these?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I relied on git ls-remote to fetch commits for each tag.

As far as updating, I am less concerned with being on bleeding edge for GHA compared to Ruby Gems. I'll probably come down to:

  • upgrades forced by GH runner deprecations
  • upgrades forced by requiring new feature/bug fix

@chris-olszewski chris-olszewski merged commit b907bd0 into main Apr 16, 2026
7 checks passed
@chris-olszewski chris-olszewski deleted the olszewski/pin_gha branch April 16, 2026 19:29
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.

3 participants