Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Removing --trace from the help file #16

Closed
KieranP opened this Issue · 6 comments

3 participants

@KieranP

--trace is great, but I don't want that option in the help file.

I still want it usable, but disabling the display of it should be configurable.

@KieranP

Any progress on this? I made a patch in my fork, but you wanted to make a cleaner way to do it. The app I've written will be going live soon, so this would be something nice to have.

@matschaffer

Where's your fork? I've been bumping into my own global option related issues (namely not being able to use -t).

@KieranP

I removed my fork a while back. The developer of this project no longer maintains it, and the project I was using it for got scrapped.

@KieranP

I still have the project I embedded it into though:

http://github.com/KieranP/homebrew/blob/rewrite/Library/Homebrew/support/commander/commander/runner.rb

Look for references to @hide_trace_help

@matschaffer

Thanks! Did you find any better libs for this sort of thing? Thor looked promising till I looked at the generated help page.

@jamesrwhite jamesrwhite referenced this issue from a commit
James White Adding ability to hide --trace, -t from the menu and globally enable …
…tracing via #16 (comment). Usage: Commander::Runner.instance.hide_trace_help
d796de5
@jamesrwhite jamesrwhite referenced this issue from a commit
James White Squashed commit of the following:
commit f8860afeede03f03afd87b905ecea7ddbaa84e3b
Author: James White <james.white.15@gmail.com>
Date:   Mon Feb 10 16:44:19 2014 +0000

    Add helper commands to delegates and some basic tests

commit 2c4cef0
Author: James White <james.white.15@gmail.com>
Date:   Sun Feb 9 00:48:52 2014 +0000

    Don't show --trace text in error messages when it's disabled. Also updated naming as tracing isn't just hidden from the menu it's actually completely disabled

commit d796de5
Author: James White <james.white.15@gmail.com>
Date:   Sun Feb 9 00:07:00 2014 +0000

    Adding ability to hide --trace, -t from the menu and globally enable tracing via #16 (comment). Usage: Commander::Runner.instance.hide_trace_help
fdd3945
@jamesrwhite jamesrwhite referenced this issue from a commit
James White Merge branch 'master' of github.com:jamesrwhite/commander
* 'master' of github.com:jamesrwhite/commander:
  Don't show --trace text in error messages when it's disabled. Also updated naming as tracing isn't just hidden from the menu it's actually completely disabled
  Adding ability to hide --trace, -t from the menu and globally enable tracing via #16 (comment). Usage: Commander::Runner.instance.hide_trace_help
0513a6a
@ggilder ggilder referenced this issue from a commit in ggilder/commander
James White Add ability to hide trace flags/globally enable tracing
Squashed:

    Add helper commands to delegates and some basic tests
    Don't show --trace text in error messages when it's disabled. Also updated naming as tracing isn't just hidden from the menu it's actually completely disabled
    Adding ability to hide --trace, -t from the menu and globally enable tracing via tj#16 (comment). Usage: Commander::Runner.instance.hide_trace_help

Don't show --trace text in error messages when it's disabled. Also updated naming as tracing isn't just hidden from the menu it's actually completely disabled

Squashed commit of the following:

commit 6120fe2
Author: James White <dev.jameswhite@gmail.com>
Date:   Mon Feb 10 17:03:55 2014 +0000

    Update README.md
fac058a
@ggilder ggilder referenced this issue from a commit in ggilder/commander
James White Add ability to hide trace flags/globally enable tracing
Squashed:

    Add helper commands to delegates and some basic tests
    Don't show --trace text in error messages when it's disabled. Also updated naming as tracing isn't just hidden from the menu it's actually completely disabled
    Adding ability to hide --trace, -t from the menu and globally enable tracing via tj#16 (comment). Usage: Commander::Runner.instance.hide_trace_help

Don't show --trace text in error messages when it's disabled. Also updated naming as tracing isn't just hidden from the menu it's actually completely disabled

Squashed commit of the following:

commit 6120fe2
Author: James White <dev.jameswhite@gmail.com>
Date:   Mon Feb 10 17:03:55 2014 +0000

    Update README.md
189105d
@ggilder ggilder referenced this issue from a commit in ggilder/commander
James White Add ability to hide trace flags/globally enable tracing
Adding ability to hide --trace, -t from the menu and globally enable
tracing via
tj#16 (comment)

Don't show --trace text in error messages when it's disabled.
cf4c063
@ggilder
Collaborator

Fixed in #67

@ggilder ggilder closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.