Skip to content

api/types/container: Summary.State change type to ContainerState #49991

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 15, 2025

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented May 15, 2025

ContainerState is currently an alias for string, so this should not be a disruptive change, but is a stepping-stone to make it a distinct type in future.

Relates to b811829

- Human readable description for the release notes

Go SDK: `api/types/container`: change `Summary.State` to a `ContainerState`.

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

ContainerState is currently an alias for string, so this should not
be a disruptive change, but is a stepping-stone to make it a distinct
type in future.

Relates to b811829

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 28.2.0 milestone May 15, 2025
@thaJeztah thaJeztah added area/api status/2-code-review impact/changelog area/go-sdk Changes affecting the Go SDK impact/go-sdk Noteworthy (compatibility changes) in the Go SDK labels May 15, 2025
@thaJeztah thaJeztah requested a review from vvoland May 15, 2025 11:41
@thaJeztah thaJeztah merged commit e7ccfae into moby:master May 15, 2025
159 checks passed
@thaJeztah thaJeztah deleted the container_summary_state branch May 15, 2025 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api area/go-sdk Changes affecting the Go SDK impact/changelog impact/go-sdk Noteworthy (compatibility changes) in the Go SDK status/2-code-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants