diff --git a/cli/command/builder/cmd.go b/cli/command/builder/cmd.go index e00cd7052..1f9b627ea 100644 --- a/cli/command/builder/cmd.go +++ b/cli/command/builder/cmd.go @@ -26,7 +26,7 @@ func newBuilderCommand(dockerCLI command.Cli) *cobra.Command { Annotations: map[string]string{"version": "1.31"}, } cmd.AddCommand( - NewPruneCommand(dockerCLI), + newPruneCommand(dockerCLI), // we should have a mechanism for registering sub-commands in the cli/internal/commands.Register function. //nolint:staticcheck // TODO: Remove when migration to cli/internal/commands.Register is complete. (see #6283) image.NewBuildCommand(dockerCLI), diff --git a/cli/command/builder/prune.go b/cli/command/builder/prune.go index 2e57aa2fd..088950eb3 100644 --- a/cli/command/builder/prune.go +++ b/cli/command/builder/prune.go @@ -33,7 +33,14 @@ type pruneOptions struct { } // NewPruneCommand returns a new cobra prune command for images +// +// Deprecated: Do not import commands directly. They will be removed in a future release. func NewPruneCommand(dockerCli command.Cli) *cobra.Command { + return newPruneCommand(dockerCli) +} + +// newPruneCommand returns a new cobra prune command for images +func newPruneCommand(dockerCLI command.Cli) *cobra.Command { options := pruneOptions{filter: opts.NewFilterOpt()} cmd := &cobra.Command{ @@ -41,14 +48,14 @@ func NewPruneCommand(dockerCli command.Cli) *cobra.Command { Short: "Remove build cache", Args: cli.NoArgs, RunE: func(cmd *cobra.Command, args []string) error { - spaceReclaimed, output, err := runPrune(cmd.Context(), dockerCli, options) + spaceReclaimed, output, err := runPrune(cmd.Context(), dockerCLI, options) if err != nil { return err } if output != "" { - fmt.Fprintln(dockerCli.Out(), output) + _, _ = fmt.Fprintln(dockerCLI.Out(), output) } - fmt.Fprintln(dockerCli.Out(), "Total reclaimed space:", units.HumanSize(float64(spaceReclaimed))) + _, _ = fmt.Fprintln(dockerCLI.Out(), "Total reclaimed space:", units.HumanSize(float64(spaceReclaimed))) return nil }, Annotations: map[string]string{"version": "1.39"}, diff --git a/cli/command/builder/prune_test.go b/cli/command/builder/prune_test.go index c798c0ab2..841e4d2cf 100644 --- a/cli/command/builder/prune_test.go +++ b/cli/command/builder/prune_test.go @@ -19,7 +19,7 @@ func TestBuilderPromptTermination(t *testing.T) { return nil, errors.New("fakeClient builderPruneFunc should not be called") }, }) - cmd := NewPruneCommand(cli) + cmd := newPruneCommand(cli) cmd.SetOut(io.Discard) cmd.SetErr(io.Discard) test.TerminatePrompt(ctx, t, cmd, cli)