Switch environment flag from camelCase to kebab-case

pivotal · Jan 28, 2022 · 0e85aa4 · 0e85aa4
1 parent 682006f
commit 0e85aa4
Show file tree

Hide file tree

Showing 9 changed files with 16 additions and 16 deletions.
diff --git a/commands/bosh.go b/commands/bosh.go
@@ -12,7 +12,7 @@ package commands
 
 type BoshCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env        EnvReader

diff --git a/commands/cf_login.go b/commands/cf_login.go
@@ -16,7 +16,7 @@ import (
 
 type CFLoginCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env           EnvReader

diff --git a/commands/om.go b/commands/om.go
@@ -12,7 +12,7 @@ package commands
 
 type OMCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env      EnvReader

diff --git a/commands/open.go b/commands/open.go
@@ -16,7 +16,7 @@ import (
 
 type OpenCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 	Show            bool   `short:"s" long:"show" description:"only show the credentials"`
 

diff --git a/commands/pks_login.go b/commands/pks_login.go
@@ -16,7 +16,7 @@ import (
 
 type PKSLoginCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env            EnvReader

diff --git a/commands/ssh.go b/commands/ssh.go
@@ -33,14 +33,14 @@ func (e *environmentName) UnmarshalFlag(name string) error {
 
 type SSHCommand struct {
 	TargetConfig    targetConfigPath     `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName environmentName      `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName environmentName      `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	Director        SSHDirectorCommand   `command:"director"`
 	OpsManager      SSHOpsManagerCommand `command:"opsman"`
 }
 
 type SSHDirectorCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env       EnvReader
@@ -61,7 +61,7 @@ func (c *SSHDirectorCommand) Execute(args []string) error {
 
 type SSHOpsManagerCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env       EnvReader

diff --git a/commands/sshuttle.go b/commands/sshuttle.go
@@ -12,7 +12,7 @@ package commands
 
 type SshuttleCommand struct {
 	TargetConfig    string `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" hidden:"true"`
-	EnvironmentName string `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
+	EnvironmentName string `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" hidden:"true"`
 	File            bool   `short:"f" long:"file" description:"write a script file but do not run it"`
 
 	Env            EnvReader

diff --git a/integration/base_command_test.go b/integration/base_command_test.go
@@ -32,10 +32,10 @@ var _ = Describe("CLI", func() {
 				Say(`Usage:`),
 				Say(`hammer \[OPTIONS\] <command>`),
 				Say(`Application Options:`),
-				Say(`-t, --target=          path to the target environment config`),
-				Say(`-e, --environmentName= name of the environment in the config to target`),
+				Say(`-t, --target=           path to the target environment config`),
+				Say(`-e, --environment-name= name of the environment in the config to target`),
 				Say(`Help Options:`),
-				Say(`-h, --help             Show this help message`),
+				Say(`-h, --help              Show this help message`),
 				Say(`Available commands:`),
 				Say(`bosh        display BOSH credentials, or run a BOSH command`),
 				Say(`cf-login    log in to the cf for the environment`),
@@ -61,10 +61,10 @@ var _ = Describe("CLI", func() {
 				Say(`Usage:`),
 				Say(`hammer \[OPTIONS\] <command>`),
 				Say(`Application Options:`),
-				Say(`-t, --target=          path to the target environment config`),
-				Say(`-e, --environmentName= name of the environment in the config to target`),
+				Say(`-t, --target=           path to the target environment config`),
+				Say(`-e, --environment-name= name of the environment in the config to target`),
 				Say(`Help Options:`),
-				Say(`-h, --help             Show this help message`),
+				Say(`-h, --help              Show this help message`),
 				Say(`Available commands:`),
 				Say(`bosh        display BOSH credentials, or run a BOSH command`),
 				Say(`cf-login    log in to the cf for the environment`),

diff --git a/main.go b/main.go
@@ -76,7 +76,7 @@ type options struct {
 	Version         versionCommand             `command:"version" alias:"ver" description:"version of command"`
 	Completion      commands.CompletionCommand `command:"completion" description:"command completion script"`
 	TargetConfig    targetConfigPath           `short:"t" long:"target" env:"HAMMER_TARGET_CONFIG" description:"path to the target environment config"`
-	EnvironmentName environmentName            `short:"e" long:"environmentName" env:"HAMMER_ENVIRONMENT_NAME" description:"name of the environment in the config to target"`
+	EnvironmentName environmentName            `short:"e" long:"environment-name" env:"HAMMER_ENVIRONMENT_NAME" description:"name of the environment in the config to target"`
 }
 
 func main() {