Merge this into nconf #1

Open
indexzero opened this Issue Jun 16, 2011 · 3 comments

Projects

None yet

2 participants

@indexzero

@bmeck ... Good work here, I think this is simple enough and core enough to warrant that it be pulled into nconf itself. No rush, this is clearly meeting our needs right now.

Let me know your thoughts.

@bmeck
Owner
bmeck commented Jun 16, 2011

Sounds good. Should it be a different kind of config provider or
something else you think? Might want to change the overrides and
defaults to also be able to accept nconf providers. We can talk about
this later.

On Thu, Jun 16, 2011 at 3:07 PM, indexzero
reply@reply.github.com
wrote:

@bmeck ... Good work here, I think this is simple enough and core enough to warrant that it be pulled into nconf itself. No rush, this is clearly meeting our needs right now.

Let me know your thoughts.

Reply to this email directly or view it on GitHub:
#1

@indexzero

@bmeck In the 0.3.0 tag of nconf I have added these features along with a more robust merge method and some helpers for merging files.

Lets take a look at where we're using reconf, update it for nconf@0.3.0 and kill this repo?

@bmeck
Owner
bmeck commented Aug 28, 2011

ok

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