-
Notifications
You must be signed in to change notification settings - Fork 385
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(go.mod): update golang.org/x/crypto to v0.31.0, et al #3334
Conversation
🛠 PR Checks Summary🔴 Maintainers must be able to edit this pull request (more info) Manual Checks (for Reviewers):
Read More🤖 This bot helps streamline PR reviews by verifying automated checks and providing guidance for contributors and reviewers. ✅ Automated Checks (for Contributors):🔴 Maintainers must be able to edit this pull request (more info) ☑️ Contributor Actions:
☑️ Reviewer Actions:
📚 Resources:Debug
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
9ab7f74
to
9ed6cfe
Compare
Updates: * golang.org/x/crypto to v0.31.0 from v0.26.0 in lieu of https://pkg.go.dev/vuln/GO-2024-3321 * golang.org/x/sync to v0.10.0 from v0.8.0 * golang.org/x/term to v0.27.0 from v0.23.0
9ed6cfe
to
e09f801
Compare
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.
LGTM. Confirmed that this PR is specific to gnolang.org/x/*
dependencies.
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.
Hi @odeke-em, these changes look great, but our review bot has detected that the permissions on your branch and how up-to-date it is needs to be fixed. Can you fix these issues so that we can integrate this PR?
Specifically:
Please re-open the PR solving conflicts and from a personal account, so we can merge in master if necessary, thank you. |
Updates: