search

mikeal / node.couchapp.js

Utility for writing couchapps.
Apache License 2.0
406 stars 78 forks source link

Fixed error when calling 'couchapp' without args #83

Closed finsterdexter closed 10 years ago

finsterdexter commented 10 years ago

I was getting an error when running couchapp on my windows vm, so I added a check that process.argv[2] is not null:

C:\>couchapp

C:\Users\finsterdexter\AppData\Roaming\npm\node_modules\couchapp\bin.js:64
  return _isUsingDirectoryConfig = (process.argv[2].trim() === "-dc");
                                                    ^
TypeError: Cannot call method 'trim' of undefined
    at isUsingDirectoryConfig (C:\Users\George\AppData\Roaming\npm\node_modules\couchapp\bin.js:64:53)
    at Object.<anonymous> (C:\Users\George\AppData\Roaming\npm\node_modules\couchapp\bin.js:81:7)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:902:3
finsterdexter commented 10 years ago

Oops, just noticed there is already a pull request for this.

Is this thing still being maintained?

mikeal commented 10 years ago

where is the other PR?

finsterdexter commented 10 years ago

https://github.com/mikeal/node.couchapp.js/pull/80

They didn't give it the most descriptive title...