Skip to content

api/types/container: deprecate ExecOptions.Detach#50218

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_execconfig_detach
Jun 18, 2025
Merged

api/types/container: deprecate ExecOptions.Detach#50218
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_execconfig_detach

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

relates to;

This field was added in 5130fe5, which added it for use as intermediate struct when parsing CLI flags (through runconfig.ParseExec) in c786a8e.

Commit 9d9dff3 rewrote the CLI to use Cobra, and as part of this introduced a separate execOptions type in api/client/container.

- Human readable description for the release notes

api/types/container: deprecate `ExecOptions.Detach`. This field is not used, and will be removed in a future release.

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

This field was added in 5130fe5, which
added it for use as intermediate struct when parsing CLI flags (through
`runconfig.ParseExec`) in c786a8e.

Commit 9d9dff3 rewrote the CLI to use
Cobra, and as part of this introduced a separate `execOptions` type in
`api/client/container`.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants