Skip to content
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

some CLI options remarks #265

Open
teto opened this issue Mar 28, 2024 · 1 comment
Open

some CLI options remarks #265

teto opened this issue Mar 28, 2024 · 1 comment

Comments

@teto
Copy link

teto commented Mar 28, 2024

just wanted to emphasize some low hanging fruits in terms of UX. I wont mention the emojis, I find them distracting but why not if the maintainers like them :)

  • Recently I tried the interactive mode:
🚀 ℹ️ [deploy] [INFO] Are you sure you want to deploy these profiles?
> yes

it doesn't suggest the answer, one has to guess what is expected. yes worked on my first try but

🚀 ℹ️ [deploy] [INFO] Are you sure you want to deploy these profiles?
(yes/no)>

could be nicer

  • --auto-rollback <auto-rollback> Override if a rollback should be attempted if activation fails
    shoul mention what to use, yes/no, true/false
  • I was going to mention the absence of units but someone already done it In which units --confirm-timeout is specified? #228
  • same for -interactive-sudo <interactive-sudo> Prompt for sudo password during activation

Actually for booleans like this, ain't it simpler to have just -interactive-sudo toggles it on ? when I had the error, I was surprised and didn't know what to put after -interactive-sudo. Same remark for the other flags --magic-rollback etc.

Thanks for the great tool, love the rollback

@PhilTaken
Copy link
Collaborator

I agree with all of those! The cli could use a little polish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants