You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are two options in baremetal defined as enums:
create.Flags().StringP(
"ipv6",
"i",
"",
"(optional) Whether IPv6 is enabled on the server. Possible values: 'yes', 'no'. Defaults to 'no'.",
)
and
create.Flags().StringP(
"notify",
"n",
"",
`(optional) Whether an activation email will be sent when the server is ready. Possible values: 'yes', 'no'.
Defaults to 'yes'.`,
)
I wonder why they are not defined as BoolP options. Is it intentional?
Since second option default value is yes, it would probably make sense to have --suppress-notification so users.
Also I wonder what is breaking changes policy for vultr-cli?
If it makes sense, i could create a PR.
The text was updated successfully, but these errors were encountered:
zikalino
changed the title
[BUG] Two options which could probably be boolean instead of enum
[Feature] Two options which could probably be boolean instead of enum
Oct 17, 2024
Describe the bug
There are two options in baremetal defined as enums:
and
Defaults to 'yes'.`,
)
I wonder why they are not defined as BoolP options. Is it intentional?
Since second option default value is yes, it would probably make sense to have --suppress-notification so users.
Also I wonder what is breaking changes policy for vultr-cli?
If it makes sense, i could create a PR.
The text was updated successfully, but these errors were encountered: