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

Improve help message when using "plonecli create" #28

Open
idgserpro opened this issue Jun 22, 2018 · 5 comments
Open

Improve help message when using "plonecli create" #28

idgserpro opened this issue Jun 22, 2018 · 5 comments

Comments

@idgserpro
Copy link

  • Plone CLI version:

0.2.0

  • Python version:

2.7.9

  • Operating System:

Description

Improve help.

What I Did

plonecli create                                                                                                                                                                 [17:14:23]
Usage: plonecli create [OPTIONS] TEMPLATE NAME
Try "plonecli create -h" for help.

Error: Missing argument "template".

And then:

plonecli create -h                                                                                                                                                              [17:14:27]
Usage: plonecli create [OPTIONS] TEMPLATE NAME

  Create a new Plone package

Options:
  -h, --help  Show this message and exit.

The first message Try "plonecli create -h" for help. is useless in this context. plonecli -l is much more useful in this case.

@kakshay21
Copy link
Member

@idgserpro Are you suggesting something like this?

plonecli create -h
Usage: plonecli create [OPTIONS] TEMPLATE_NAME PACKAGE_NAME
   
    Create a new Plone package

TEMPLATE_NAME -
   - addon
   - theme_package
   - buildout

Options:
  -h, --help  Show this message and exit.

@idgserpro
Copy link
Author

Or something in the lines of:

plonecli create -h
Usage: plonecli create [OPTIONS] TEMPLATE_NAME PACKAGE_NAME
   
    Create a new Plone package. Run "plonecli -l" to see available template names.

So your TEMPLATE_NAME would always be up to date.

@idgserpro
Copy link
Author

I think plonecli add could have the same treatment.

@mzfr
Copy link

mzfr commented Jan 13, 2019

Hi @idgserpro I would like to work on this issue.

@idgserpro
Copy link
Author

Hi @mzfr. We're not maintainers of this project, I think you can keep asking for advice on the PR you made like #41 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants