From 21fdaf11c6df6bd82bc38be0ca798d29eaf5fc86 Mon Sep 17 00:00:00 2001 From: Asaf Gabai Date: Thu, 9 Jul 2026 17:36:59 +0300 Subject: [PATCH 1/2] Remotely delete quietly on CI pipelines --- apptrust/commands/flags.go | 2 +- apptrust/commands/version/remote_delete_app_version_cmd.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/apptrust/commands/flags.go b/apptrust/commands/flags.go index 1a1c07c..ada8dfb 100644 --- a/apptrust/commands/flags.go +++ b/apptrust/commands/flags.go @@ -126,7 +126,7 @@ var flagsMap = map[string]components.Flag{ CreateRepoFlag: components.NewBoolFlag(CreateRepoFlag, "Set to true to create the repository on the edge if it does not exist.", components.WithBoolDefaultValueFalse()), MappingPatternFlag: components.NewStringFlag(MappingPatternFlag, "Specify along with "+MappingTargetFlag+" to distribute artifacts to a different path on the edge node. You can use wildcards to specify multiple artifacts.", func(f *components.StringFlag) { f.Mandatory = false }), MappingTargetFlag: components.NewStringFlag(MappingTargetFlag, "The target path for distributed artifacts on the edge node. If not specified, the artifacts will have the same path and name on the edge node, as on the source Artifactory server. For flexibility in specifying the distribution path, you can include placeholders in the form of {1}, {2} which are replaced by corresponding tokens in the pattern path that are enclosed in parenthesis.", func(f *components.StringFlag) { f.Mandatory = false }), - QuietFlag: components.NewBoolFlag(QuietFlag, "Set to true to skip the confirmation message.", components.WithBoolDefaultValueFalse()), + QuietFlag: components.NewBoolFlag(QuietFlag, "Set to true to skip the confirmation message. When $CI is true, the default value is true.", components.WithBoolDefaultValueFalse()), } var commandFlags = map[string][]string{ diff --git a/apptrust/commands/version/remote_delete_app_version_cmd.go b/apptrust/commands/version/remote_delete_app_version_cmd.go index da00fdf..7073fd8 100644 --- a/apptrust/commands/version/remote_delete_app_version_cmd.go +++ b/apptrust/commands/version/remote_delete_app_version_cmd.go @@ -54,7 +54,7 @@ func (rd *remoteDeleteAppVersionCommand) prepareAndRunCommand(ctx *components.Co rd.applicationKey = ctx.Arguments[0] rd.version = ctx.Arguments[1] - rd.quiet = ctx.GetBoolFlagValue(commands.QuietFlag) + rd.quiet = pluginsCommon.GetQuietValue(ctx) if err := ValidateDistributionFlags(ctx); err != nil { return err From cc89ef49c34316974c1666a0de12f9b2b7129b64 Mon Sep 17 00:00:00 2001 From: Asaf Gabai Date: Thu, 9 Jul 2026 17:46:37 +0300 Subject: [PATCH 2/2] Remotely delete quietly on CI pipelines --- .../commands/version/remote_delete_app_version_cmd_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/apptrust/commands/version/remote_delete_app_version_cmd_test.go b/apptrust/commands/version/remote_delete_app_version_cmd_test.go index 77f2900..f83d6e7 100644 --- a/apptrust/commands/version/remote_delete_app_version_cmd_test.go +++ b/apptrust/commands/version/remote_delete_app_version_cmd_test.go @@ -13,6 +13,7 @@ import ( "github.com/jfrog/jfrog-cli-application/apptrust/model" "github.com/jfrog/jfrog-cli-core/v2/plugins/components" "github.com/jfrog/jfrog-cli-core/v2/utils/config" + "github.com/jfrog/jfrog-cli-core/v2/utils/coreutils" "github.com/stretchr/testify/assert" "github.com/stretchr/testify/require" ) @@ -170,6 +171,8 @@ func TestRemoteDeleteAppVersionCommand_QuietSuite(t *testing.T) { for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { + t.Setenv(coreutils.CI, "false") + ctrl := gomock.NewController(t) defer ctrl.Finish()