From deff01c4c7ddb0750eea2054ab1d4af7bc92949a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Pereira?= Date: Thu, 8 Aug 2024 15:54:48 -0500 Subject: [PATCH] Change order of flags so they are alphabetized MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: João Pereira --- command/v7/rollback_command.go | 2 +- integration/v7/isolated/rollback_command_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/command/v7/rollback_command.go b/command/v7/rollback_command.go index abda4d5e4c4..14854c42a36 100644 --- a/command/v7/rollback_command.go +++ b/command/v7/rollback_command.go @@ -17,11 +17,11 @@ type RollbackCommand struct { Force bool `short:"f" description:"Force rollback without confirmation"` RequiredArgs flag.AppName `positional-args:"yes"` + MaxInFlight *int `long:"max-in-flight" description:"Defines the maximum number of instances that will be actively being rolled back."` Strategy flag.DeploymentStrategy `long:"strategy" description:"Deployment strategy can be canary or rolling. When not specified, it defaults to rolling."` Version flag.Revision `long:"version" required:"true" description:"Roll back to the specified revision"` relatedCommands interface{} `related_commands:"revisions"` usage interface{} `usage:"CF_NAME rollback APP_NAME [--version VERSION] [-f]"` - MaxInFlight *int `long:"max-in-flight" description:"Defines the maximum number of instances that will be actively being rolled back."` LogCacheClient sharedaction.LogCacheClient Stager shared.AppStager diff --git a/integration/v7/isolated/rollback_command_test.go b/integration/v7/isolated/rollback_command_test.go index e862edb55e1..056c2115b9e 100644 --- a/integration/v7/isolated/rollback_command_test.go +++ b/integration/v7/isolated/rollback_command_test.go @@ -35,9 +35,9 @@ var _ = Describe("rollback command", func() { Expect(session).To(Say(`cf rollback APP_NAME \[--version VERSION\]`)) Expect(session).To(Say("OPTIONS:")) Expect(session).To(Say(`-f\s+Force rollback without confirmation`)) - Expect(session).To(Say("--strategy Deployment strategy can be canary or rolling. When not specified, it defaults to rolling.")) - Expect(session).To(Say(`--version\s+Roll back to the specified revision`)) Expect(session).To(Say("--max-in-flight")) + Expect(session).To(Say(`--strategy\s+Deployment strategy can be canary or rolling. When not specified, it defaults to rolling.`)) + Expect(session).To(Say(`--version\s+Roll back to the specified revision`)) Expect(session).To(Say("SEE ALSO:")) Expect(session).To(Say("revisions")) })