-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
vendor: github.com/docker/docker cfdca8dc1d4c (v25.0.0-dev) #4673
Conversation
No significant changes, so I'll keep this in draft for now. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #4673 +/- ##
=======================================
Coverage 59.75% 59.75%
=======================================
Files 287 287
Lines 24821 24821
=======================================
Hits 14832 14832
Misses 9103 9103
Partials 886 886 |
|
also added nolint:staticcheck to spec.Networks refs full diff: moby/moby@v25.0.0-beta.1...cfdca8d Co-authored-by: Albin Kerouanton <[email protected]> Signed-off-by: Sebastiaan van Stijn <[email protected]>
72a0f94
to
539537d
Compare
Chatted with @akerouanton and conversion is already there, so we only needed |
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!
full diff: moby/moby@v25.0.0-beta.1...cfdca8d
- What I did
- How I did it
- How to verify it
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)