Use standardised flag; not camel case (#401)

* Use standardised flag; not camel case

* Catch the error

pkg/cmd/build/new.go

@@ -15,6 +15,7 @@ import (
 	"github.com/buildkite/go-buildkite/v4"
 	"github.com/charmbracelet/huh/spinner"
 	"github.com/spf13/cobra"
+	"github.com/spf13/pflag"
 )
 
 func NewCmdBuildNew(f *factory.Factory) *cobra.Command {
@@ -99,7 +100,10 @@ func NewCmdBuildNew(f *factory.Factory) *cobra.Command {
 	cmd.Flags().StringArrayVarP(&env, "env", "e", []string{}, "Set environment variables for the build")
 	cmd.Flags().BoolVarP(&ignoreBranchFilters, "ignore-branch-filters", "i", false, "Ignore branch filters for the pipeline")
 	cmd.Flags().BoolVarP(&confirmed, "yes", "y", false, "Skip the confirmation prompt. Useful if being used in automation/CI")
-	cmd.Flags().StringVarP(&envFile, "envFile", "f", "", "Set the environment variables for the build via an environment file")
+	cmd.Flags().StringVarP(&envFile, "env-file", "f", "", "Set the environment variables for the build via an environment file")
+	cmd.Flags().StringVarP(&envFile, "envFile", "", "", "Set the environment variables for the build via an environment file")
+	_ = cmd.Flags().MarkDeprecated("envFile", "use --env-file instead")
+	cmd.Flags().SetNormalizeFunc(normaliseFlags)
 	cmd.Flags().SortFlags = false
 	return &cmd
 }
@@ -145,3 +149,11 @@ func newBuild(ctx context.Context, org string, pipeline string, f *factory.Facto
 
 	return util.OpenInWebBrowser(web, build.WebURL)
 }
+
+func normaliseFlags(pf *pflag.FlagSet, name string) pflag.NormalizedName {
+	switch name {
+	case "envFile":
+		name = "env-file"
+	}
+	return pflag.NormalizedName(name)
+}