Skip to content

Conversation

@jsternberg
Copy link
Collaborator

No description provided.

@jedevc
Copy link
Collaborator

jedevc commented Jan 5, 2024

cc @thaJeztah since updates to docker/cli 😄

@thaJeztah
Copy link
Member

Overall looks fine, but before merging, you could consider having a look at moby/buildkit#4523 - if that's merged, you could update moby and docker/cli to v25.0.0-rc.1

The tricky bit is that neither docker/cli nor moby/moby are a module yet, so currently it requires "manual" compare of the vendor.mod from these repositories; one dependency that looks to be "behind" is Cobra; in docker/cli we updated to 1.8. CI looks green so it's not critical, and I'm happy to do a follow-up to align some f the remaining modules.

@thaJeztah
Copy link
Member

@jsternberg moby/buildkit#4523 was merged, so you can consider updating to current master 👍

Signed-off-by: Jonathan A. Sternberg <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

LGTM (nb)

@tonistiigi tonistiigi merged commit 9060cab into docker:master Jan 5, 2024
@jsternberg jsternberg deleted the vendor-update branch January 5, 2024 20:50
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.

4 participants