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


##########
pkg/cmd/reset.go:
##########
@@ -56,6 +56,21 @@ type resetCmdOptions struct {
 }
 
 func (o *resetCmdOptions) reset(cmd *cobra.Command, _ []string) {
+       if cmd.InOrStdin() == os.Stdin {

Review Comment:
   I think it would be better to include a `-f/--force` option to skip the 
check entirely. As shown by the e2e test it is possible that some user leverage 
the command automatically, so we need some possibility to skip the check and 
force the reset without prompting. In that case, you can change the test to 
pass a `-f` and make it pass.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to