cli icon indicating copy to clipboard operation
cli copied to clipboard

[Feature Request] Standardize Command Option string-enum value format

Open yuandrew opened this issue 1 year ago • 1 comments

Is your feature request related to a problem? Please describe.

Some use camel case (type has values like FirstWorkflowTask), others use snake case (reject-condition has not_open), some capitalize the first letter (reapply-type has All), others are all lowercase (wait-for-stage has accepted). This is likely a product of history.

Describe the solution you'd like

We should land on a standardized format for this. But we also need to make sure we support aliasing the old options for backwards compat

Additional context

yuandrew avatar Sep 24 '24 16:09 yuandrew

log-format needs to be changed to string-enum, but pretty needs to be a undocumented legacy alias

yuandrew avatar Sep 24 '24 18:09 yuandrew

This would be a nice to have, but there are global temporal values here that would be changed, and it's a larger scope to address, and not something that's necessarily worth it at the moment. Maybe worth revisiting once tctl has been archived, that might make this easier.

yuandrew avatar Nov 19 '24 17:11 yuandrew