Skip to content

vendor: github.com/docker/docker v28.3.0-dev (6a1fb46d4805) #6131

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

Merged
merged 1 commit into from
Jun 13, 2025

Conversation

vvoland
Copy link
Collaborator

@vvoland vvoland commented Jun 13, 2025

full diff: moby/moby@v28.2.2...6a1fb46

Signed-off-by: Paweł Gronowski pawel.gronowski@docker.com

@vvoland vvoland added this to the 28.3.0 milestone Jun 13, 2025
@vvoland vvoland self-assigned this Jun 13, 2025
@vvoland vvoland requested a review from thaJeztah June 13, 2025 16:03
@codecov-commenter
Copy link

codecov-commenter commented Jun 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.04%. Comparing base (3ce5130) to head (4cb0695).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6131   +/-   ##
=======================================
  Coverage   55.04%   55.04%           
=======================================
  Files         361      361           
  Lines       30152    30152           
=======================================
  Hits        16596    16596           
  Misses      12598    12598           
  Partials      958      958           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@thaJeztah
Copy link
Member

Oh! Looks like something is still using one of the old events fields;

69.59 cli/command/container/utils.go:78:10: SA1019: e.Status is deprecated: use Action instead. Information from JSONMessage. With data only in container events. (staticcheck)
69.59 		switch e.Status {
69.59 		       ^

full diff: moby/moby@v28.2.2...6a1fb46

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
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.

LGTM

@vvoland vvoland merged commit f53bb88 into docker:master Jun 13, 2025
87 checks passed
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