Skip to content

Update index.js#1

Merged
AlexAllocated merged 1 commit into
yargs:masterfrom
martinheidegger:patch-1
Dec 6, 2013
Merged

Update index.js#1
AlexAllocated merged 1 commit into
yargs:masterfrom
martinheidegger:patch-1

Conversation

@martinheidegger

Copy link
Copy Markdown
Contributor

When check() fails a error occurs. It seems to be fixed with this small change. Unit tests would be good ...

ReferenceError: defaults is not defined
at /Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:339:46
at Array.forEach (native)
at Object.Argv.self.help (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:302:14)
at Object.Argv.self.showHelp (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:241:17)
at fail (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:168:18)
at /Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:420:17
at Array.forEach (native)
at parseArgs (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:410:16)
at Object.Argv.Object.defineProperty.get as argv
at Object. (/Users/heideggermartin/IdeaProjects/mapslice/bin/mapslice:24:46)

When check() fails a error occurs. It seems to be fixed with this small change. Unit tests would be good ...

ReferenceError: defaults is not defined
    at /Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:339:46
    at Array.forEach (native)
    at Object.Argv.self.help (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:302:14)
    at Object.Argv.self.showHelp (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:241:17)
    at fail (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:168:18)
    at /Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:420:17
    at Array.forEach (native)
    at parseArgs (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:410:16)
    at Object.Argv.Object.defineProperty.get [as argv] (/Users/heideggermartin/IdeaProjects/mapslice/node_modules/yargs/index.js:368:36)
    at Object.<anonymous> (/Users/heideggermartin/IdeaProjects/mapslice/bin/mapslice:24:46)
@AlexAllocated

Copy link
Copy Markdown

Sorry it took me a couple days. I was busy and saved the email notification then promptly forgot about it. Thank you for the fix.

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.

2 participants