rockcarver / frodo-cli

A CLI to manage ForgeRock platform deployments supporting Identity Cloud tenants, ForgeOps deployments, and classic deployments.
MIT License
17 stars 16 forks source link

Consistent arg name handling #23

Closed CharanMann closed 1 year ago

CharanMann commented 2 years ago

Currently some args names are optional and others are mandatory, refer below:

These args should be passed with arg names such as:

This will avoid any confusion regarding these args handling

vscheuber commented 2 years ago

@CharanMann we introduced this behavior with rockcarver/frodo#6 "Refactor command options into arguments". We wanted to move away from using named params for mandatory params that had to be there always to cut back on number of characters to type. The commands I use now are streamlined like this:

frodo journey list volker The above command uses volker as the host argument and reads the connection profile and auto-populates the username and password arguments. The realm argument is using the determined default value depending on detected deployment type: /alpha for cloud, / for forgeops and classic.

For users using frodo every day, every character saved helps.

atomicsamurai commented 1 year ago

Testing slack integration.

vscheuber commented 1 year ago

Won't do. Re-open if you see a strong need to change our use of arguments and options.