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.

Sign up for GitHub

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

Jump to bottom

Pass the original error as the second argument of fail() handler. (fixes #373) #382

Merged
merged 1 commit into from
Feb 14, 2016
Merged

Pass the original error as the second argument of fail() handler. (fixes #373) #382

merged 1 commit into from
Feb 14, 2016

Conversation

gajus
Copy link
Contributor

@gajus gajus commented Feb 14, 2016

A follow up to #374

@gajus gajus mentioned this pull request Feb 14, 2016
Closed
bcoe added a commit that referenced this pull request Feb 14, 2016
@bcoe
Merge pull request #382 from gajus/stack-trace-v4
496f4e7 
Pass the original error as the second argument of fail() handler. (fixes #373)
@bcoe bcoe merged commit 496f4e7 into yargs:4.x Feb 14, 2016
@bcoe
Copy link
Member

bcoe commented Feb 14, 2016

@gajus awesome, thanks for the contribution ✋

This was referenced Mar 26, 2016
Closed
Merged
bcoe pushed a commit that referenced this pull request Apr 4, 2016
@nexdrew @bcoe
docs(cleanup): Add missing features to README docs
066972b 
* document additional command module props a la #414
* document Error instance for fail handler a la #382
* add missing locales to readme
* add "number" to en.json locale
* add "number" to it.json locale
@nexdrew nexdrew mentioned this pull request May 20, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gajus @bcoe