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

Hash-pin workflow Actions #189

Merged
merged 3 commits into from Jun 28, 2023
Merged

Hash-pin workflow Actions #189

merged 3 commits into from Jun 28, 2023

Conversation

pnacht
Copy link
Contributor

@pnacht pnacht commented Jun 27, 2023

Fixes #188.

As I mentioned in the issue, this PR hash-pins all workflow Actions and sets up dependabot to keep them updated. For examples of dependabot in action, see the PRs in my fork.

Currently dependabot will only monitor the Actions, since the project doesn't seem to have other types of dependencies. If I missed something and you want me to set up dependabot to keep other dependencies up to date, let me know and I'll gladly patch the PR.

Signed-off-by: Pedro Kaj Kjellerup Nacht <pnacht@google.com>
Signed-off-by: Pedro Kaj Kjellerup Nacht <pnacht@google.com>
Signed-off-by: Pedro Kaj Kjellerup Nacht <pnacht@google.com>
@thockin
Copy link
Contributor

thockin commented Jun 27, 2023

LGTM - @pohly, any issues?

@pohly
Copy link
Contributor

pohly commented Jun 28, 2023

It makes sense. I'm a bit worried about constantly having to merge dependabot PRs (it's been a nuisance and led to maintainer fatigue elsewhere), but we can deal with that if it becomes a problem.

@pohly pohly merged commit 0ad2c4b into go-logr:master Jun 28, 2023
15 checks passed
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.

Hash-pin workflow dependencies
3 participants