Skip to content

pkg/chrootarchive: use stdlib errors, remove "// import" comments#49151

Merged
AkihiroSuda merged 2 commits intomoby:masterfrom
thaJeztah:chrootarchive_reduce_deps
Dec 20, 2024
Merged

pkg/chrootarchive: use stdlib errors, remove "// import" comments#49151
AkihiroSuda merged 2 commits intomoby:masterfrom
thaJeztah:chrootarchive_reduce_deps

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

relates to:

pkg/chrootarchive: use stdlib errors

Remove use of github.com/pkg/errors for this package, in preparation
of it being moved to a separate module.

pkg/chrootarchive: remove "// import" comments

These comments were added to prevent users from accidentally
importing using the wrong module name, but they don't have
an effect when working in go modules mode.

Remove the comments in preparation of moving this package
to a separate module.

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

Remove use of github.com/pkg/errors for this package, in preparation
of it being moved to a separate module.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
These comments were added to prevent users from accidentally
importing using the wrong module name, but they don't have
an effect when working in go modules mode.

Remove the comments in preparation of moving this package
to a separate module.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code area/go-sdk labels Dec 19, 2024
@thaJeztah thaJeztah added this to the 28.0.0 milestone Dec 19, 2024
@thaJeztah thaJeztah self-assigned this Dec 19, 2024
@AkihiroSuda AkihiroSuda merged commit 2c56497 into moby:master Dec 20, 2024
@thaJeztah thaJeztah deleted the chrootarchive_reduce_deps branch December 20, 2024 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/go-sdk 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