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

Fixed issue where it was treating 'help' as the verb that usage info was... #2

Closed
ericnewton76 opened this issue Nov 4, 2017 · 7 comments

Comments

@ericnewton76
Copy link
Member

Issue by tombatron
Thursday Apr 04, 2013 at 01:55 GMT
Originally opened as gsscoder/commandline#74


... requested for.

Let's say that I have an application called 'example' that has a verb called 'dothing'. If I entered the following on the console:

example help dothing

The usage text displayed isn't for the verb specified but the default usage text.


tombatron included the following code: https://github.com/gsscoder/commandline/pull/74/commits

@ericnewton76
Copy link
Member Author

Comment by gsscoder
Thursday Apr 04, 2013 at 13:24 GMT


Hi,
you're right the correct behavior should the one described on the top.

Thanks for reporting and for the fix.

Sorry but for the moment I'm unable to accept PR (as stated in gsscoder/commandline#67).

This will be of great help in my current 2.0 development (for the moment still unpublished); since it's a deep kernel rewrite, you can understand, that in the transitory phase this would mean maintain two branches...

The fix will be integrated in the new branch. :D

Thanks,
Giacomo Stelluti Scala

@ericnewton76
Copy link
Member Author

Comment by tombatron
Thursday Apr 04, 2013 at 13:30 GMT


Awesome!

@ericnewton76
Copy link
Member Author

Comment by jonnyyu
Friday Sep 13, 2013 at 11:09 GMT


Hi can we take a few minutes to merge this PR?
The issue is really a deal breaker for the Verb commands feature in stable 1.9.71.2. Who can use Verb commands without verb specific help?

Since it already took 5 months to develop the 2.0 since then, maybe it will take another few months. Before the 2.0 is ready, can we just take a few time let current version work?

After all, this is really a one-line fix.

Thanks!

@ericnewton76
Copy link
Member Author

Comment by nemec
Friday Sep 13, 2013 at 23:20 GMT


@jonnyyu this pull request can't be merged as-is since the codebase has changed radically since then. Are you still experiencing this problem?

Can you upgrade to the latest version from git (2.0) and see if it's still an issue?

On the project homepage there's a "download zip" button on the bottom right. Just extract the project folder and add it as a reference in your solution. Only @gsscoder has NuGet privileges, so I can't help there.

@ericnewton76
Copy link
Member Author

Comment by jonnyyu
Monday Sep 23, 2013 at 15:40 GMT


No, I finally gave up using it and switch to ManyConsole instead.

On Sat, Sep 14, 2013 at 7:20 AM, Dan Nemec [email protected] wrote:

@jonnyyu https://github.com/jonnyyu this pull request can't be merged
as-is since the codebase has changed radically since then. Are you still
experiencing this problem?


Reply to this email directly or view it on GitHubhttps://github.com/gsscoder/commandline/pull/74#issuecomment-24430970
.

Yingshen(Jonny) Yu

@ericnewton76
Copy link
Member Author

Comment by JMerrill
Monday Jan 19, 2015 at 14:09 GMT


How different is the ManyConsole API? How long did it take you to convert from CommandLine? Thanks for any info.

@moh-hassan
Copy link
Collaborator

Resolved.
For verb help, run:

    myapp  myverb --help 

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

No branches or pull requests

2 participants