Skip to content

chore: Pin Github Actions to Commit Hashes (NOBUMP)#36

Merged
btrautmann merged 2 commits into
mainfrom
frizbee-updates-20250701172259
Jul 3, 2025
Merged

chore: Pin Github Actions to Commit Hashes (NOBUMP)#36
btrautmann merged 2 commits into
mainfrom
frizbee-updates-20250701172259

Conversation

@6f6d6172

@6f6d6172 6f6d6172 commented Jul 1, 2025

Copy link
Copy Markdown
Contributor

This PR includes automated changes from running frizbee to pin Github Actions. For more information on what this actually means, take a look at our documentation on UnpinnedActions. Because frizbee will be replacing tags with their corresponding commit hash, this change should fundamentally be a no-op. This change just makes our code more explicit about what we're using.

Note that because we're touching Github Workflow files in this PR, there is a chance that Claws, our GHA Static Analyzer, will find other issues in those files that were introduced before this PR was created. These findings will need to be addressed before this PR can be merged.

The Claws documentation has a good list for how to remediate each type of finding, but for brevity, here are some of the commonly seen ones:

  • RiskyTriggers: Fires when a workflow has workflow_dispatch or pull_request_target; remediate by leaving comments explaning why, and put an ignore statement (example)
  • UnsafeCheckout: Fires when a workflow checks out user supplied code; remediate by leaving a comment explaining how the user supplied code is used and confirm it isn't executed (example)

For other findings and how to remediate them, check the Claws docs!

@6f6d6172 6f6d6172 requested a review from a team as a code owner July 1, 2025 21:23
@btrautmann btrautmann changed the title chore: Pin Github Actions to Commit Hashes chore: Pin Github Actions to Commit Hashes (NOBUMP) Jul 3, 2025

@btrautmann btrautmann 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.

domain lgtm
platform lgtm

@btrautmann btrautmann merged commit 5376a9e into main Jul 3, 2025
6 of 8 checks passed
@btrautmann btrautmann deleted the frizbee-updates-20250701172259 branch July 3, 2025 19: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