Skip to content

[improvement] Use new picocli lib for CLI#24

Merged
iamdanfox merged 1 commit into
developfrom
fo/better-cli
Oct 25, 2018
Merged

[improvement] Use new picocli lib for CLI#24
iamdanfox merged 1 commit into
developfrom
fo/better-cli

Conversation

@ferozco
Copy link
Copy Markdown
Contributor

@ferozco ferozco commented Oct 25, 2018

Similar to palantir/conjure-java#91. Replace usage of commons cli with picocli

ex:

Usage: conjure-python generate [-hV] [--rawSource] [--writeCondaRecipe] [--packageDescription=<packageDescription>]
                               [--packageName=<packageName>] [--packageUrl=<packageUrl>]
                               [--packageVersion=<packageVersion>] <input> <output>
Generate Python bindings for a Conjure API
      <input>              Path to the input IR file
      <output>             Output directory for generated source
      --packageDescription=<packageDescription>
                           The description of the package to generate.
      --packageName=<packageName>
                           The name of the package to generate.
      --packageUrl=<packageUrl>
                           The url of the package to generate
      --packageVersion=<packageVersion>
                           The version of the package to generate.
      --rawSource          Only generate the plain source without any package metadata
      --writeCondaRecipe   Only generate the plain source without any package metadata
  -h, --help               Show this help message and exit.
  -V, --version            Print version information and exit.

Copy link
Copy Markdown
Contributor

@iamdanfox iamdanfox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

picocli is so great!

@iamdanfox iamdanfox merged commit 05805eb into develop Oct 25, 2018
@iamdanfox iamdanfox deleted the fo/better-cli branch October 25, 2018 15:01
derenrich pushed a commit to derenrich/conjure-python that referenced this pull request Jan 8, 2019
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