-
Notifications
You must be signed in to change notification settings - Fork 192
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
CLI command to show all confirguration options #4260
Comments
Mainly I guess it would be just pretty printing:
|
Note that the current design was intentional and mirrors the interface of |
well I'm not convinced that is necessarily the best part of the git CLI lol. But anyway, if we want to mirror that then (a) it has a list option |
I am not sure, but your assessment of the current status of
But that is not true, it shows the options that are specifically set for that profile. However, I see your point that it might be useful to see the defaults of certain options and their description from the CLI, and agree that we should not stick to the original git design at all costs if we really think it is not optimal. So I am happy to consider an alternative. In that case, we may also want to think how to include the configuration for the caching, which I haven't included under Maybe we should first draw up the new envisioned interface and sign-off on it before starting to implement, in order to prevent length back and forths on a PR. |
folding in to #4584 |
Currently, unless I am mistaken, there is no way to quickly view all configuration options for a profile:
verdi profile show
has an options line, but only shows options specifically set as global overrides,verdi config
has auto-complete of configuration names, but does not show their current values.It would be nice to be able to see all the configuration values for a profile without having to search through the code for the defaults.
Perhaps
verdi config
should have an extra level, i.e.verdi config set
andverdi config show
, and there could beverdi config show --all
, with a*
to denote values that override defaults, e.g.The text was updated successfully, but these errors were encountered: