-
Notifications
You must be signed in to change notification settings - Fork 902
feat(actions_permissions): sha_pinning_required #2870
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
base: main
Are you sure you want to change the base?
feat(actions_permissions): sha_pinning_required #2870
Conversation
Signed-off-by: Leonard Sheng Sheng Lee <leonard.sheng.sheng.lee@gmail.com>
6289202 to
49ad201
Compare
| github.com/golangci/unconvert v0.0.0-20240309020433-c5143eacb3ed // indirect | ||
| github.com/google/go-cmp v0.6.0 // indirect | ||
| github.com/google/go-cmp v0.7.0 // indirect | ||
| github.com/google/go-github/v78 v78.0.0 // indirect |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately this provider is on v67 and the upgrade has not been made to get us to v78 yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nickfloyd, thanks for the review. 🙇
Shall I keep this PR open while waiting for it to be upgraded?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, let's do that... I'm hopeful we'll make some good progress getting there. This is the batch where we are tracking that, please feel free to take on any of the version bumps.
We are currently trying to move toward v68 - but the breaking changes in that one are significant - all of the old project APIs were removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait until google/go-github#3807 is being included in the next release.Resolves #2869.
Before the change?
sha_pinning_requiredoption for actions permissions is not supported.After the change?
sha_pinning_requiredoption actions permissions is supported.Pull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!