You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/04/18 13:51:08 UTC

[GitHub] [camel-k] squakez commented on a diff in pull request #3219: fix: kamel run -o incompatible with --dev

squakez commented on code in PR #3219:
URL: https://github.com/apache/camel-k/pull/3219#discussion_r852130408


##########
pkg/cmd/run.go:
##########
@@ -243,6 +243,10 @@ func (o *runCmdOptions) validate() error {
 		return err
 	}
 
+	if o.OutputFormat != "" && o.Dev {
+		return errors.New("option '-o' is not compatible with '--dev'")

Review Comment:
   We better keep using the `fmt` package as we're doing for the rest of errors, ie `return fmt.Errorf(invalid label specification %s. Expected "<labelkey>=<labelvalue>", label)`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org