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

suggestions for cli #2259

Closed
el-tocino opened this issue Aug 19, 2019 · 4 comments
Closed

suggestions for cli #2259

el-tocino opened this issue Aug 19, 2019 · 4 comments
Labels
help wanted Type: Enhancement - proposed New proposal for a feature that is not currently a priority on the roadmap.

Comments

@el-tocino
Copy link
Contributor

el-tocino commented Aug 19, 2019

A :config or :configuration option, to list/show/point to the running configuration. Useful for finding odd config bits easily.

Also handy would be an option to disable audio output when queried from the CLI. (:mute?) This would theoretically be configurable as not everyone will want it to be quiet.

Lastly, a less verbose mode would be nice. Something like a :verbose option perhaps to turn on/off log display. This would NOT effect mycroft logging, just the display.

@penrods
Copy link
Contributor

penrods commented Sep 5, 2019

I'm about done with a standalone mycroft-config utility that does something like this:

mycroft-config:  Mycroft configuration manager
usage: mycroft-config [COMMAND] [params]

COMMANDs:
  edit (system|user)                  edit and validate config file
  reload                              instruct services to reload configs
  show (default|remote|system|user)   display the specified setting file
  set <var>                           set the variable (under USER)
  get [var]                           display a particular variable
                                      or all if no 'var' specified
Note: Use jq format for specifying <var>

Examples:
  mycroft-config edit user
  sudo mycroft-config edit system
  mycroft-config show remote
  mycroft-config get
  mycroft-config get enclosure.platform
  mycroft-config set test.subvalue "foo"

Does this sound adequate? It also wouldn't be difficult to add access to this inside the CLI if preferred.

@el-tocino
Copy link
Contributor Author

A most welcome addition indeed.

@penrods
Copy link
Contributor

penrods commented Sep 6, 2019

See PR #2290 for the mycroft-config tool. Comments welcomed!

@adocampo
Copy link
Contributor

adocampo commented Sep 12, 2019

Sounds awesome. To be completely perfect I would suggest auto-completion with double Tab to help users to navigate through the options (I mean the possible options inside the json), so:
mycrot-co[tab-tab]nfig set encl[tab-tab]osure.[tab-tab]
As you can see, the auto-complete would show all the options inside enclosure...

@krisgesling krisgesling added Type: Enhancement - proposed New proposal for a feature that is not currently a priority on the roadmap. and removed type: feature labels Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Type: Enhancement - proposed New proposal for a feature that is not currently a priority on the roadmap.
Projects
None yet
Development

No branches or pull requests

4 participants