Skip to content

Conversation

@thaJeztah
Copy link
Member

no code changes, only updated swagger (docs)

full diff: moby/moby@v28.0.0-rc.2...62bc597

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah
Copy link
Member Author

We should look if we can move the swagger file to a directory that doesn't have Go code, so that it doesn't land in the go module 🤔

@codecov-commenter
Copy link

codecov-commenter commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.16%. Comparing base (a8f8886) to head (fe349e6).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5840   +/-   ##
=======================================
  Coverage   59.16%   59.16%           
=======================================
  Files         352      352           
  Lines       29567    29567           
=======================================
  Hits        17493    17493           
  Misses      11092    11092           
  Partials      982      982           

no code changes, only updated swagger (docs)

full diff: moby/moby@v28.0.0-rc.2...00ab386

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@vvoland vvoland force-pushed the bump_engine_28.0_rc3 branch from c2b60b9 to fe349e6 Compare February 18, 2025 22:05
@vvoland vvoland marked this pull request as ready for review February 18, 2025 22:05
@vvoland
Copy link
Collaborator

vvoland commented Feb 18, 2025

Updated after merging moby/moby#49491

@vvoland vvoland merged commit 1dcfd23 into docker:master Feb 18, 2025
89 checks passed
@thaJeztah thaJeztah deleted the bump_engine_28.0_rc3 branch February 18, 2025 22:22
@thaJeztah thaJeztah added this to the 28.0.0 milestone Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants