Skip to content

api/types/network: move network options from api to client#50786

Merged
thaJeztah merged 4 commits intomoby:masterfrom
austinvazquez:move-network-options-from-api-to-client
Aug 22, 2025
Merged

api/types/network: move network options from api to client#50786
thaJeztah merged 4 commits intomoby:masterfrom
austinvazquez:move-network-options-from-api-to-client

Conversation

@austinvazquez
Copy link
Contributor

- What I did
Partial of #50740

- How I did it
Moves the network.ListOptions and network.InspectOptions types to the client mod

- How to verify it

- Human readable description for the release notes

api/types/network: move the `ListOptions` and `InspectOptions` types to the client

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

@austinvazquez austinvazquez added this to the 29.0.0 milestone Aug 22, 2025
@austinvazquez austinvazquez added area/api API impact/api kind/refactor PR's that refactor, or clean-up code area/go-sdk release-blocker PRs we want to block a release on labels Aug 22, 2025
@austinvazquez austinvazquez self-assigned this Aug 22, 2025
@austinvazquez austinvazquez force-pushed the move-network-options-from-api-to-client branch from 68ac1ba to 72e491e Compare August 22, 2025 02:36
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 after squashing that commit 😅

@thaJeztah
Copy link
Member

oh! also needs a rebase now 😅

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
@austinvazquez austinvazquez force-pushed the move-network-options-from-api-to-client branch from 72e491e to fe8516c Compare August 22, 2025 14:39
@austinvazquez
Copy link
Contributor Author

@thaJeztah, I rebased and squashed the commit if you can take another look thanks! :)

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

sorry thought I did, but I had the tab still open 😂

@thaJeztah thaJeztah merged commit b3a701a into moby:master Aug 22, 2025
321 of 324 checks passed
@austinvazquez austinvazquez deleted the move-network-options-from-api-to-client branch August 23, 2025 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api API area/go-sdk impact/api kind/refactor PR's that refactor, or clean-up code release-blocker PRs we want to block a release on status/4-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants