From dea4050d322475ed9d849f6513b1e9206e206daf Mon Sep 17 00:00:00 2001 From: Filip Rudy Date: Thu, 23 Jan 2025 10:53:46 +0100 Subject: [PATCH 1/2] add succesful operation message after manage/unmanage --- internal/cmd/alpha/module/manage.go | 3 +++ internal/cmd/alpha/module/unmanage.go | 4 ++++ 2 files changed, 7 insertions(+) diff --git a/internal/cmd/alpha/module/manage.go b/internal/cmd/alpha/module/manage.go index 405c7d6f8..b37ab71b2 100644 --- a/internal/cmd/alpha/module/manage.go +++ b/internal/cmd/alpha/module/manage.go @@ -59,5 +59,8 @@ func runManage(cfg *manageConfig) clierror.Error { if err != nil { return clierror.Wrap(err, clierror.New("failed to check module state")) } + + fmt.Printf("Module %s set to managed\n\n", cfg.module) + return nil } diff --git a/internal/cmd/alpha/module/unmanage.go b/internal/cmd/alpha/module/unmanage.go index c15708a32..b4568a587 100644 --- a/internal/cmd/alpha/module/unmanage.go +++ b/internal/cmd/alpha/module/unmanage.go @@ -1,6 +1,7 @@ package module import ( + "fmt" "github.com/kyma-project/cli.v3/internal/clierror" "github.com/kyma-project/cli.v3/internal/cmdcommon" "github.com/spf13/cobra" @@ -46,5 +47,8 @@ func runUnmanage(cfg *unmanageConfig) clierror.Error { if err != nil { return clierror.Wrap(err, clierror.New("failed to check module state")) } + + fmt.Printf("Module %s set to unmanaged\n\n", cfg.module) + return nil } From 59fcc68251990a58513f3ef0a7af2119a3c09ccf Mon Sep 17 00:00:00 2001 From: Filip Rudy Date: Thu, 23 Jan 2025 13:13:29 +0100 Subject: [PATCH 2/2] remove unnecessary newlines --- internal/cmd/alpha/module/manage.go | 2 +- internal/cmd/alpha/module/unmanage.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/cmd/alpha/module/manage.go b/internal/cmd/alpha/module/manage.go index b37ab71b2..676c17d45 100644 --- a/internal/cmd/alpha/module/manage.go +++ b/internal/cmd/alpha/module/manage.go @@ -60,7 +60,7 @@ func runManage(cfg *manageConfig) clierror.Error { return clierror.Wrap(err, clierror.New("failed to check module state")) } - fmt.Printf("Module %s set to managed\n\n", cfg.module) + fmt.Printf("Module %s set to managed\n", cfg.module) return nil } diff --git a/internal/cmd/alpha/module/unmanage.go b/internal/cmd/alpha/module/unmanage.go index b4568a587..fbed29dbc 100644 --- a/internal/cmd/alpha/module/unmanage.go +++ b/internal/cmd/alpha/module/unmanage.go @@ -48,7 +48,7 @@ func runUnmanage(cfg *unmanageConfig) clierror.Error { return clierror.Wrap(err, clierror.New("failed to check module state")) } - fmt.Printf("Module %s set to unmanaged\n\n", cfg.module) + fmt.Printf("Module %s set to unmanaged\n", cfg.module) return nil }