Skip to content

ci: add permissions and pin all actions to full commit SHAs#771

Closed
XananasX7 wants to merge 1 commit into
libtom:developfrom
XananasX7:ci/pin-actions
Closed

ci: add permissions and pin all actions to full commit SHAs#771
XananasX7 wants to merge 1 commit into
libtom:developfrom
XananasX7:ci/pin-actions

Conversation

@XananasX7

Copy link
Copy Markdown

main.yml was missing a top-level permissions: block and used mutable action tag references.

Add permissions: contents: read (least privilege).

Pin all uses: to full commit SHAs (tags kept as comments):

Action Before After
actions/checkout @v4 @34e114876b0b11c390a56381ad16ebd13914f8d5
actions/upload-artifact @v4 @ea165f8d65b6e75b540449e92b4886f43607fa02
nick-fields/retry @v4.0.0 @ad984534de44a9489a53aefd81eb77f87c70dc60

Qualifies under the Google Open Source Patch Rewards Program.

  permissions: contents: read added (was missing)
  actions/checkout: @v4 -> @34e114876b0b11c390a56381ad16ebd13914f8d5
  actions/upload-artifact: @v4 -> @ea165f8d65b6e75b540449e92b4886f43607fa02
  nick-fields/retry: @v4.0.0 -> @ad984534de44a9489a53aefd81eb77f87c70dc60
@sjaeckel sjaeckel closed this Jun 14, 2026
@sjaeckel

Copy link
Copy Markdown
Member

Sorry, but no.

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