-
Notifications
You must be signed in to change notification settings - Fork 223
Bump c/storage to v1.59.0, c/image to v5.36.0 #2479
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
Bump c/storage to v1.59.0, c/image to v5.36.0 #2479
Conversation
Reviewer's GuideUpgrade primary container modules (containers/image and containers/storage) to the latest releases and sync all indirect dependencies via go mod tidy, updating go.sum accordingly. File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Hey @TomSweeneyRedHat - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sourcery-ai[bot], TomSweeneyRedHat The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
7e00216 to
7aa32e5
Compare
Possible replacement for containers#2476. I'm not sure if I messed something up when trying to merge in containers#2478 there, so thought I'd create this PR to see if I messed something up there when doing the merge/vendor. Signed-off-by: tomsweeneyredhat <[email protected]>
7aa32e5 to
5e7264f
Compare
Bump c/common to v0.64.0 in preparation for Podman v5.6 Signed-off-by: tomsweeneyredhat <[email protected]>
Bump to the next dev version, 0.65.0-dev, in the main c/common branch. Signed-off-by: tomsweeneyredhat <[email protected]>
|
Don't merge! I was working on a few small lint errors when the prior PR was closed. I'll create a new PR to fix those up. |
|
Closing. Versions and lint issues will be taken care of elsewhere. |
Possible replacement for #2476. I'm not sure if I messed something up when trying to merge in #2478 there, so I thought I'd create this PR to see if I messed something up there when doing the merge/vendor.
If this passes the lint tests, I'll add the common bumps, and I'll kill #2476
Summary by Sourcery
Upgrade primary and indirect Go dependencies to the latest compatible versions
Enhancements:
Build: