Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Pin actions to SHAs for security and reproducibility #165

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

zkoppert
Copy link
Contributor

@zkoppert zkoppert commented Jun 11, 2024

Pull Request

Proposed Changes

This pull request pins the versions of GitHub actions used in the project's workflows. The versions of these actions have been pinned to specific commits for increased stability and security. Version tags can be moved and create unpredictable results, so pinning to a specific commit ensures that the same version of the action is used every time the workflow runs.

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run npm run lint and fix any linting issues that have been introduced
  • run npm run test and run tests
  • If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

  • Label as either bug, documentation, enhancement, infrastructure, maintenance, or breaking

@zkoppert zkoppert requested a review from a team as a code owner June 11, 2024 23:27
Copy link
Contributor

@ajhenry ajhenry left a comment

Choose a reason for hiding this comment

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

Good call, thanks!

Copy link
Contributor

@jmeridth jmeridth left a comment

Choose a reason for hiding this comment

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

@jmeridth jmeridth merged commit c3d7727 into main Jun 12, 2024
13 checks passed
@jmeridth jmeridth deleted the action-pinning branch June 12, 2024 01:32
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