cli/command: deprecate DockerCli.Apply
The Apply method was added when CLI options for constructing the CLI were rewritten into functional options in [cli@7f207f3]. There was no mention in the pull request of this method specifically, and this may have been related to work being done elsewhere on compose-on-kubernetes or the compose-cli plugin that may have needed options to modify the CLI config after it was already initialized. The CLI itself no longer depends on this method since [cli@133279f], and the only known consumer (docker compose) no longer needs it since [cli@2711800] and [cli@048e931]. This patch deprecates the method with the intent to remove it in a future release. [cli@7f207f3]:7f207f3f95[cli@133279f]:133279fb0d[cli@2711800]:2711800430[cli@048e931]:048e931b42Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This commit is contained in:
@ -567,6 +567,8 @@ func (cli *DockerCli) initialize() error {
|
||||
}
|
||||
|
||||
// Apply all the operation on the cli
|
||||
//
|
||||
// Deprecated: this method is no longer used and will be removed in the next release if there are no remaining users.
|
||||
func (cli *DockerCli) Apply(ops ...CLIOption) error {
|
||||
for _, op := range ops {
|
||||
if err := op(cli); err != nil {
|
||||
|
||||
Reference in New Issue
Block a user