Skip to content

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

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

client/image_(inspect,history,load,save): Wrap return values#51236
thaJeztah merged 1 commit intomoby:masterfrom
vvoland:client-image-opts-functionalwrap

Conversation

@vvoland
Copy link
Contributor

@vvoland vvoland commented Oct 20, 2025

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
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

@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 Distribution 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