client/image_(inspect,history,load,save): Wrap return values by vvoland · Pull Request #51236 · moby/moby · GitHub
Skip to content

client/image_(inspect,history,load,save): Wrap return values - #51236

Merged
thaJeztah merged 1 commit into
moby:masterfrom
vvoland:client-image-opts-functionalwrap
Oct 21, 2025
Merged

thaJeztah merged 1 commit into
moby:masterfrom
vvoland:client-image-opts-functionalwrap

Conversation

@vvoland

@vvoland vvoland commented Oct 20, 2025 •

Copy link
Copy Markdown
Contributor

This still keeps them with the functional arguments and only wraps the return values.

`client`: Wrap return values of `ImageInspect`, `ImageHistory`, `ImageLoad` and `ImageSave` in a struct.

@vvoland vvoland added this to the 29.0.0 milestone Oct 20, 2025
@vvoland vvoland self-assigned this Oct 20, 2025
@vvoland vvoland added kind/refactor PR's that refactor, or clean-up code release-blocker PRs we want to block a release on labels Oct 20, 2025
@vvoland vvoland added impact/changelog impact/go-sdk Noteworthy (compatibility changes) in the Go SDK labels Oct 20, 2025
@vvoland
vvoland force-pushed the client-image-opts-functionalwrap branch from 50f90c2 to 598a10e Compare October 20, 2025 21:44
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztah force-pushed the client-image-opts-functionalwrap branch from 598a10e to 2d69edd Compare October 21, 2025 12:24

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
thaJeztah merged commit ac9838f into moby:master Oct 21, 2025
181 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dependencies area/images Image Service impact/changelog impact/go-sdk Noteworthy (compatibility changes) in the Go SDK kind/refactor PR's that refactor, or clean-up code module/client release-blocker PRs we want to block a release on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants