Skip to content

CLI Usability Tweeks#654

Merged
underscorephil merged 2 commits into
softlayer:masterfrom
sudorandom:usability-tweaks
Dec 4, 2015
Merged

CLI Usability Tweeks#654
underscorephil merged 2 commits into
softlayer:masterfrom
sudorandom:usability-tweaks

Conversation

@sudorandom
Copy link
Copy Markdown
Contributor

  • Removes global --debug option in favor of --verbose | -v. -v can be specified multiple times to increase verbosity. For example, -vvv will the same as --debug=3
  • Removes global --timings option
  • Renames global --fixtures option to --demo
  • Adds more diagnostics whenever the verbose (-v) flag is set including the info that was displayed with --timings
  • Converts to using lowercase name/value for tables instead of upper-case since lower-case is generally preferred for this tool at the moment
  • Shows default values for many CLI options

@sudorandom
Copy link
Copy Markdown
Contributor Author

The new pylint errors will be solved by #655

Removes global `--debug` option in favor of `--verbose` | `-v`.
Removes global `--timings` option
Adds more diagnostics whenever the verbose (-v) flag is set including the info that was displayed with `--timings`
Converts to using lowercase name/value for tables instead of upper-case since lower-case is generally perferred for this tool at the moment
Shows default values for many CLI options
@sudorandom sudorandom assigned sudorandom and unassigned sudorandom Dec 3, 2015
@sudorandom
Copy link
Copy Markdown
Contributor Author

This change is good to review.

@sudorandom sudorandom added this to the 4.2.0 milestone Dec 3, 2015
@sudorandom sudorandom changed the title Usability Tweeks CLI Usability Tweeks Dec 3, 2015
@underscorephil
Copy link
Copy Markdown
Contributor

+1

underscorephil added a commit that referenced this pull request Dec 4, 2015
@underscorephil underscorephil merged commit 0ba294e into softlayer:master Dec 4, 2015
@sudorandom sudorandom deleted the usability-tweaks branch December 4, 2015 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants