Nuwk icon indicating copy to clipboard operation
Nuwk copied to clipboard

Support project-wide configuration

Open mackwic opened this issue 9 years ago • 6 comments

I am still uncertain about the greatest name convention for this (maybe throw it in the package.json ? But which one ?)

Anyway, we need a project-wide configuration. Defaults will be the one shipped with the application, and each value will be overridable.

Proposals for the local config file:

  • .nuwk
  • .config.nuwk
  • ??

Sparse config file are bad. It clutters the file tree with thing we never edit again. I am all for checking first the dedicated config file, then check for the nuwk key in the package.json, then apply defaults.

What do you think of that ?

mackwic avatar Jul 01 '14 10:07 mackwic

I think it fits. About the name i guess .config.nuwk would be more obvious.

micc83 avatar Jul 01 '14 10:07 micc83

So be it.

mackwic avatar Jul 01 '14 10:07 mackwic

node-runtime-configuration seems to handle the job quiet well, with environment and command lime overloading if necessary.

It will look at .nuwkrc files, though. Rc standing for "runtime configuration". It's ok for me, what do you think ? By the way, I am pinging the owner of the repo, just to see if he's responsive. If not, I'll fork and we can adapt the project to our needs.

mackwic avatar Jul 04 '14 22:07 mackwic

Hi @mackwic, any new from the owner?

micc83 avatar Jul 07 '14 13:07 micc83

Yep, he’s waiting for my pull request, which is on its way. Then we’ll just have to nom install his package.

It will probably be ok wednesday.

--   Thomas Wickham [email protected]

De: Alessandro Benoit [email protected] Répondre: micc83/Nuwk [email protected] Date: 7 juillet 2014 at 15:32:53 À: micc83/Nuwk [email protected] Cc: Thomas Wickham [email protected] Sujet:  Re: [Nuwk] Support project-wide configuration (#5)

Hi @mackwic, any new from the owner?

— Reply to this email directly or view it on GitHub.

mackwic avatar Jul 07 '14 13:07 mackwic

Cool!

2014-07-07 15:34 GMT+02:00 Thomas Wickham [email protected]:

Yep, he’s waiting for my pull request, which is on its way. Then we’ll just have to nom install his package.

It will probably be ok wednesday.

Thomas Wickham [email protected]

De: Alessandro Benoit [email protected] Répondre: micc83/Nuwk [email protected] Date: 7 juillet 2014 at 15:32:53 À: micc83/Nuwk [email protected] Cc: Thomas Wickham [email protected] Sujet: Re: [Nuwk] Support project-wide configuration (#5)

Hi @mackwic, any new from the owner?

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/micc83/Nuwk/issues/5#issuecomment-48178330.

Alessandro Benoit Cel: +39 392 0078370 Email: [email protected]

micc83 avatar Jul 07 '14 13:07 micc83