Skip to content

Add ability to specify arguments syntax for top-level command#258

Merged
SomeKittens merged 1 commit intotj:masterfrom
rrthomas:args-for-top-level
Mar 19, 2015
Merged

Add ability to specify arguments syntax for top-level command#258
SomeKittens merged 1 commit intotj:masterfrom
rrthomas:args-for-top-level

Conversation

@rrthomas
Copy link
Contributor

@rrthomas rrthomas commented Sep 9, 2014

  • index.js: Add arguments method, support top-level command in action.

This would fix issue #257.

@thethomaseffect
Copy link
Collaborator

Could you add some tests for this?

* index.js: Add arguments method, support top-level command in action.
@rrthomas
Copy link
Contributor Author

Done.

@goloroden
Copy link

Any chance that this will be merged in the near future?

@SomeKittens
Copy link
Collaborator

LGTM. Sorry about the wait.

SomeKittens added a commit that referenced this pull request Mar 19, 2015
Add ability to specify arguments syntax for top-level command
@SomeKittens SomeKittens merged commit 5091eac into tj:master Mar 19, 2015
@rrthomas
Copy link
Contributor Author

Thanks, @SomeKittens!

@goloroden
Copy link

Awesome, I really appreciate what you did here :-)

The only thing I am missing is an update to the docs: How do I use this?

@rrthomas
Copy link
Contributor Author

@goloroden: for now, see #257. Please file a new issue for documentation, and I'll do it when I can.

@goloroden
Copy link

Okay :-)

@zhiyelee
Copy link
Collaborator

zhiyelee commented Apr 8, 2015

@thethomaseffect @rrthomas IMO, It would be better that PRs like this should get merged after the document is ready.

I am preparing for the next release and find this PR without document.

@rrthomas rrthomas deleted the args-for-top-level branch April 8, 2015 17:22
@rrthomas
Copy link
Contributor Author

rrthomas commented Apr 8, 2015

@zhiyelee see PR #392.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants