f3ath / cider

Tools for Dart package maintainers
https://pub.dev/packages/cider
MIT License
101 stars 17 forks source link

`bump` and `log` have no subcommands #69

Closed marvin-kolja closed 7 months ago

marvin-kolja commented 8 months ago

Description

Currently, cider bump <part> and cider log <type> are not subcommands. Thus, something like cider bump --help doesn't print the correct information.

You can take a look at how the change looks like here: https://github.com/marvin-kolja/cider/commit/473acd1786e0ed614294c793a43a11d9bf782b6f

Current Behaviour

$ dart bin/cider.dart bump --help
Bump project version

Usage: cider bump [arguments]
-h, --help               Print this usage information.
    --[no-]keep-build    Keep the existing build
    --[no-]bump-build    Also bump the build
    --build              Sets the build to the given value
                         (defaults to "")
    --pre                Sets the pre-release to the given value
                         (defaults to "")

Run "cider help" to see global options.

Wanted behaviour

$ dart bin/cider.dart bump --help
Bump project version

Usage: cider bump <subcommand> [arguments]
-h, --help    Print this usage information.

Available subcommands:
  breaking   Bump the breaking version
  build      Bump the build version
  major      Bump the major version
  minor      Bump the minor version
  patch      Bump the patch version
  pre        Bump the pre-release version
  release    Bump the release version

Run "cider help" to see global options.
$ dart bin/cider.dart bump breaking --help
Bump the breaking version

Usage: cider bump breaking [arguments]
-h, --help               Print this usage information.
    --[no-]keep-build    Keep the existing build
    --[no-]bump-build    Also bump the build
    --build              Sets the build to the given value
                         (defaults to "")
    --pre                Sets the pre-release to the given value
                         (defaults to "")

Run "cider help" to see global options.
f3ath commented 7 months ago

Thank you. The change you're suggesting seems to be a nice UX improvement, wanna open a PR?

marvin-kolja commented 7 months ago

Fantastic, I'll do that.