Skip to content

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

Closed

Description

@ericnewton76

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

Activity

  1. ericnewton76 commented on Nov 4, 2017

    @ericnewton76
    MemberAuthor

    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

  2. ericnewton76 commented on Nov 4, 2017

    @ericnewton76
    MemberAuthor

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


    Awesome!

  3. ericnewton76 commented on Nov 4, 2017

    @ericnewton76
    MemberAuthor

    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!

  4. ericnewton76 commented on Nov 4, 2017

    @ericnewton76
    MemberAuthor

    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.

  5. ericnewton76 commented on Nov 4, 2017

    @ericnewton76
    MemberAuthor

    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 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/gsscoder/commandline/pull/74#issuecomment-24430970
    .

    Yingshen(Jonny) Yu

  6. ericnewton76 commented on Nov 4, 2017

    @ericnewton76
    MemberAuthor

    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.

  7. moh-hassan commented on Oct 9, 2019

    @moh-hassan
    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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions