pushkin-consortium / pushkin-cli

A CLI designed to facilitate the setup process of a Pushkin stack
MIT License
0 stars 4 forks source link

Verbose flag #118

Closed jessestorbeck closed 1 year ago

jessestorbeck commented 1 year ago

Resolves #109. Adds default "quiet mode" for install site, install experiment, prep, and start. Use --verbose flag for additional output.

Additionally, experiment templates for lexical decision and grammaticality judgment are added back to the options in install experiment.

hunterschep commented 1 year ago

Testing the 'quiet' CLI:

A couple small things:

jessestorbeck commented 1 year ago

@hunterschep

The console seems to log/spit out '--verbose flag set inside someFunction()' a lot, I don't know if this was intended?

Yes, this was intentional, but we can talk about whether @jkhartshorne still wants this in verbose mode.

When quiet mode is set, especially on longer commands where there is simply nothing it could be a nice feature to have a progress percentage [XX%] like node/yarn/brew have featured as to let the user know that the cli is not hung on something.

Completely agree, but there is no simple way (from what I could tell) of shoehorning a meaningful progress bar into the CLI. I looked into cli-progress and some other similar packages, and it all seemed to require manually incrementing the progress bar and/or restructuring the code to fit the format expected by the package. There doesn't seem to be any way to drop a few lines into the existing CLI and have it automatically implement the progress bar in a meaningful way. We could easily enough add a bar that went 0% --> 50% --> 100%, but I don't know if that's much better than nothing at all. If you have any ideas, feel free to make a suggestion.