dominictarr / rc

The non-configurable configuration loader for lazy people.
Other
1.02k stars 97 forks source link

update minimist #55

Closed ralphtheninja closed 9 years ago

ralphtheninja commented 9 years ago

rc is using an old version of minimist and I'd like to be able to use minimist aliases, e.g.

var minimist = require('minimist')

module.exports = require('rc')('myapp', {
  foo: 'foovalue',
  bar: 'barvalue'
}, minimist(process.argv, {
  alias: { foo: 'f', bar: 'b' }
}))

While I can always do this it's nice if rc uses the latest minimist so npm can dedupe properly whenever a dependant depends on both rc and minimist.

Alternatively I'd like to be able to somehow pass the aliases object into rc which in turn passes it to minimist.

ralphtheninja commented 9 years ago

Basically it's the dedupe step I'm after :)

dominictarr commented 9 years ago

okay cool

dominictarr commented 9 years ago

i decided that the correct thing to do is make this a minor change. merged into 1.1.0

ralphtheninja commented 9 years ago

:+1: