Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Set cli.notFoundUsage to false to not show `Cannot find module ...` #65
Conversation
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
What does this do? I don't really see the value. |
indexzero
closed this
Jun 2, 2012
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
pksunkara
Jun 2, 2012
Contributor
@indexzero, I am giving a use case below,
I have a flatiron cli tool named hub. I have several commands in that tool which are in here. Now, when a user gives some arguments for the tool, I want flatiron to do all the initial stuff and load the proper command. If the command is not found, I want the tool to spawn git [arguments]
and pipe the output. Without this pr, flatiron first prints some help before spawning git
. This commit enables us to not to display the Cannot find module ...
message
This is important for my tool
@indexzero, I am giving a use case below, I have a flatiron cli tool named hub. I have several commands in that tool which are in here. Now, when a user gives some arguments for the tool, I want flatiron to do all the initial stuff and load the proper command. If the command is not found, I want the tool to spawn This is important for my tool |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
indexzero
Jun 2, 2012
Owner
Hmm ... ok. I see. I'll reopen it. Merge away. Please write some docs and tests for this kind of stuff. We're really lacking good test fixtures in core.
Hmm ... ok. I see. I'll reopen it. Merge away. Please write some docs and tests for this kind of stuff. We're really lacking good test fixtures in core. |
indexzero
reopened this
Jun 2, 2012
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
Merged |
pksunkara commentedJun 1, 2012
I needed this thing for a small flatiron cli app I was writing.