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... #74

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

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

wants to merge 1 commit into from

Conversation

tombatron
Copy link

... 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.

@gsscoder
Copy link
Owner

gsscoder commented Apr 4, 2013

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 #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

@tombatron
Copy link
Author

Awesome!

@jonnyyu
Copy link

jonnyyu commented Sep 13, 2013

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!

@nemec
Copy link
Collaborator

nemec commented Sep 13, 2013

@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.

@jonnyyu
Copy link

jonnyyu commented Sep 23, 2013

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

On Sat, Sep 14, 2013 at 7:20 AM, Dan Nemec notifications@github.com 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//pull/74#issuecomment-24430970
.

Yingshen(Jonny) Yu

@JMerrill
Copy link

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

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.

5 participants