feat: remove default value for revision flag

Author: JGiola

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+### Changed
+
+- remove default value for revision flag in the deploy command
+
 ### Added
 
 - new command for listing the user companies

docs/30_commands.md

@@ -115,4 +115,4 @@ Available flags for the command:
 - `--no-semver`, to force the deploy without `semver`
 - `--insecure`, to skip certificate check
 - `--project-id`, to set the project ID from command line
-- `--revision`, to specify the revision of the commit to deploy (default `HEAD`)
+- `--revision`, to specify the revision of the commit to deploy

docs/40_examples.md

@@ -55,13 +55,13 @@ The deploy command allow you to trigger a new deploy pipeline for the current pr
 argument needed is the environment id that you want to deploy:
 
 ```sh
-miactl deploy development
+miactl deploy development --revision main
 ```
 
 Additionally if your context don’t contain the project id you can select it via a flag:
 
 ```sh
-miactl deploy development --project-id <your-project-id>
+miactl deploy development --project-id <your-project-id> --revision main
 ```
 
 You can customize the way your project is deployed:

internal/clioptions/clioptions.go

@@ -66,7 +66,11 @@ func (f *CLIOptions) AddCompanyFlags(cmd *cobra.Command) {
 }
 
 func (f *CLIOptions) AddDeployFlags(cmd *cobra.Command) {
-	cmd.PersistentFlags().StringVar(&f.Revision, "revision", "HEAD", "Revision of the commit to deploy")
+	cmd.PersistentFlags().StringVar(&f.Revision, "revision", "", "Revision of the commit to deploy")
+	if err := cmd.MarkPersistentFlagRequired("revision"); err != nil {
+		// if there is an error something very wrong is happening, panic
+		panic(err)
+	}
 	cmd.PersistentFlags().StringVar(&f.DeployType, "deploy-type", "smart_deploy", "Deploy type")
 	cmd.PersistentFlags().BoolVar(&f.NoSemVer, "no-semver", false, "Force the deploy wihout semver")
 }