Skip to content

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

Open
XananasX7 wants to merge 1 commit into
xiph:masterfrom
XananasX7:ci/pin-actions
Open

ci: add permissions and pin actions to full commit SHAs#909
XananasX7 wants to merge 1 commit into
xiph:masterfrom
XananasX7:ci/pin-actions

Conversation

@XananasX7

Copy link
Copy Markdown

All 4 non-fuzz workflow files (action.yml, distcheck.yml, msys2.yml, options.yml) were missing a top-level permissions: block and used mutable action tag references.

Add permissions: contents: read to all 4 workflows.

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

Action Before After
actions/checkout @v4 @34e114876b0b11c390a56381ad16ebd13914f8d5
actions/upload-artifact @v4 @ea165f8d65b6e75b540449e92b4886f43607fa02
msys2/setup-msys2 @v2 @66cd2cce69caa17b53920067426061ca1de3a884

Qualifies under the Google Open Source Patch Rewards Program.

All 4 non-fuzz workflow files were missing a top-level permissions block
and used mutable tag references. Add permissions: contents: read and
pin every uses: to its full commit SHA (tag kept as comment):

  actions/checkout: @v4 -> @34e114876b0b11c390a56381ad16ebd13914f8d5
  actions/upload-artifact: @v4 -> @ea165f8d65b6e75b540449e92b4886f43607fa02
  msys2/setup-msys2: @v2 -> @66cd2cce69caa17b53920067426061ca1de3a884
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.

1 participant