Skip to content

pkg/archive: replace uses of pkg/errors for stdlib errors#49101

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:archive_stdlib_errs
Dec 16, 2024
Merged

pkg/archive: replace uses of pkg/errors for stdlib errors#49101
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:archive_stdlib_errs

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Dec 16, 2024

Reducing dependencies in preparation of moving pkg/archive to a separate module.

Most errors were already using stdlib, and only 3 places used pkg/errors, so it was already inconsistent.

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

Reducing dependencies in preparation of moving pkg/archive to
a separate module.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Dec 16, 2024
@thaJeztah thaJeztah added this to the 28.0.0 milestone Dec 16, 2024
@thaJeztah thaJeztah self-assigned this Dec 16, 2024
@thaJeztah thaJeztah marked this pull request as ready for review December 16, 2024 12:04
@thaJeztah
Copy link
Member Author

Discussed with @crazy-max on Slack, and it doesn't look like BuildKit requires pkg/errors errors for these, so I'll bring this one in

@thaJeztah thaJeztah merged commit aa052c8 into moby:master Dec 16, 2024
@thaJeztah thaJeztah deleted the archive_stdlib_errs branch December 16, 2024 14:23
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

Development

Successfully merging this pull request may close these issues.

2 participants