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

An option description can be located after a new line #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EVGVir
Copy link
Contributor

@EVGVir EVGVir commented Aug 7, 2016

Sometimes if an option is too long, it looks better when its description starts on a new line.

Such behaviour is supported (though it is not documented) by the original docopt. A commit with corresponding test is requested to be pulled.

Sometimes if an option is too long, it looks better when its description
starts on a new line.
@jaredgrubb
Copy link
Member

I think this is a great patch. However, the testcase file is copied verbatim from the docopt-py repo. Would you mind offering the testcase-only patch to that repo first? I'm willing to do it if you dont want to. The test case actually does pass the Python version but fails the C++ version -- which I think makes it a GREAT testcase.

@EVGVir
Copy link
Contributor Author

EVGVir commented Sep 4, 2016

As I wrote above, the commit for docopt-py is awaiting his destiny. I have prepared those commits simultaneously.

Moreover, the day before yesterday I have written directly to Vladimir @keleshev about my two pull requests. But unfortunately, I have no reaction yet.

@Camille019
Copy link

Is there an update on the status of this PR? I'm also interested by this feature.

@jaredgrubb
Copy link
Member

I've posted a comment on the docopt mentioned by EVGVir. Hopefluly it'll get merged there, and then we can do it here :/

@indianakernick
Copy link
Contributor

This actually doesn't quite work properly. -f and -b are added to the map instead of --foo and --bar. I really think this feature is useful for creating beautiful command-line interfaces!

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

Successfully merging this pull request may close these issues.

4 participants