Skip to content

remove uses of github.com/docker/docker/errdefs#6079

Merged
thaJeztah merged 11 commits into
docker:masterfrom
thaJeztah:less_errdefs
May 16, 2025
Merged

remove uses of github.com/docker/docker/errdefs#6079
thaJeztah merged 11 commits into
docker:masterfrom
thaJeztah:less_errdefs

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Use local definitions that implement the correct interface instead

- Human readable description for the release notes

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

thaJeztah added 3 commits May 16, 2025 20:18
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added this to the 28.2.0 milestone May 16, 2025
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels May 16, 2025
@thaJeztah thaJeztah requested review from dmcgowan and vvoland May 16, 2025 18:21
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 16, 2025

Codecov Report

Attention: Patch coverage is 46.22642% with 57 lines in your changes missing coverage. Please review.

Project coverage is 55.02%. Comparing base (eee0cf9) to head (1058b22).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6079      +/-   ##
==========================================
- Coverage   55.06%   55.02%   -0.04%     
==========================================
  Files         359      361       +2     
  Lines       30078    30126      +48     
==========================================
+ Hits        16562    16578      +16     
- Misses      12565    12594      +29     
- Partials      951      954       +3     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

thaJeztah added 8 commits May 16, 2025 20:28
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah merged commit 0674a00 into docker:master May 16, 2025
87 checks passed
@thaJeztah thaJeztah deleted the less_errdefs branch May 16, 2025 19:06
@thaJeztah thaJeztah self-assigned this May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants