Pass in yes flag for minder cli commands that have a prompt message

[rdimitrov]

It would be useful to pass in a 'yes' prompt (or no for that matter) when performing commands that have a confirmation prompt, i.e. minder auth delete.

[hemakshis]

Hi @rdimitrov
I'd like to take up this issue. Since this is my first time here, would appreciate any help or pointers!

[JAORMX]

Hey @hemakshis ! We really appreciate you wanting to contribute! It's a little late in Europe where some of us are. But we'll get you more details tomorrow morning. Unless someone from the team works late today. Thanks again!

[rdimitrov]

Hi @rdimitrov I'd like to take up this issue. Since this is my first time here, would appreciate any help or pointers!

Hey @hemakshis 👋 This is great, thanks for reaching out! 💯

The goal of this issue is to implement a --yes/-y flag for some of the minder commands. The issue mentions also the option of adding a --no/-n flag, but let's leave that aside for now (but do share if you found a command where calling it with a --no flag would be valuable).

Here's some helpful information.

Implementation hints:

• Find all minder CLI commands that have prompts (use cli.PrintYesNoPrompt())
• Add the --yes/-y flag to each command that you found that make sense to have this as an optional flag
• Update each command implementation so it uses the value of the flag accordingly, i.e. skipping the prompts.

References:

• Get familiar with our contributing guidelines - here
• Minder uses two complementary frameworks for the CLI implementation - cobra and viper. You can check their docs/examples to get a sense of their usage.
• This is how a similar flag is added to a command - here
• This is how you can use its value - here
• This is how you can make decisions based on its value - here

Feel free to reach out if I've missed something or you have any questions 👍

[hemakshis]

Thanks @rdimitrov for such detailed info, I will try it out and let you know how it goes on Monday :)

[hemakshis]

Hi @rdimitrov
I have created #1730, please review 🙂