-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modify default value of overwrite flag in generate-bundle cmd #3280
Modify default value of overwrite flag in generate-bundle cmd #3280
Conversation
@@ -210,6 +210,6 @@ func (c *bundleCmd) addCommonFlagsTo(fs *pflag.FlagSet) { | |||
fs.StringVar(&c.crdsDir, "crds-dir", "", "Root directory for CustomResoureDefinition manifests") | |||
fs.StringVar(&c.channels, "channels", "alpha", "A comma-separated list of channels the bundle belongs to") | |||
fs.StringVar(&c.defaultChannel, "default-channel", "", "The default channel for the bundle") | |||
fs.BoolVar(&c.overwrite, "overwrite", false, "Overwrite the bundle's metadata and Dockerfile if they exist") | |||
fs.BoolVar(&c.overwrite, "overwrite", true, "Overwrite the bundle's metadata and Dockerfile if they exist") | |||
fs.BoolVarP(&c.quiet, "quiet", "q", false, "Run in quiet mode") | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing fragment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing fragment. Otherwise, all shows ok.
Could you please add this one?
26ebe23
to
f75dce7
Compare
With this PR, "generate bundle" command by default will overwrite metadata files if already present.
f75dce7
to
3ab4428
Compare
@camilamacedo86 Added changelog fragment. Please have a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Description of the change:
With this PR, "generate bundle" command by default will overwrite
metadata files if already present.