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.

Sign up for GitHub

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

Jump to bottom

gh: pin actions by commit SHA #23

Merged
merged 8 commits into from
Dec 16, 2024
Merged

gh: pin actions by commit SHA #23

merged 8 commits into from
Dec 16, 2024

Conversation

bschaatsbergen
Copy link
Contributor

@bschaatsbergen bschaatsbergen commented Dec 16, 2024 •
edited
Loading

  • Pins GitHub Actions to commit SHAs to prevent running code from newly published tags.
  • Added golangci-lint step.

@bschaatsbergen
gh: pin actions by commit SHA
c1e3855 
Pins GitHub Actions to commit SHAs to prevent running code from newly published tags.
@bschaatsbergen bschaatsbergen added the github_actions Pull requests that update GitHub Actions code label Dec 16, 2024
@bschaatsbergen
Copy link
Contributor Author

@mmourick I’ll hold off on addressing the linter errors in the code paths touched in #19 to avoid introducing conflicts. Let me know if you’d prefer otherwise.

@bschaatsbergen bschaatsbergen mentioned this pull request Dec 16, 2024
Merged
@bschaatsbergen bschaatsbergen merged commit 40a1886 into main Dec 16, 2024
1 of 2 checks passed
@bschaatsbergen bschaatsbergen deleted the gh/pin-by-sha branch December 16, 2024 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
github_actions Pull requests that update GitHub Actions code github-actions tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bschaatsbergen