vigour-io / ferry

0 stars 0 forks source link

Extract most of bin/index.js to a separate repo #15

Closed shawninder closed 9 years ago

shawninder commented 9 years ago

I like this way of handling config. I might want to use it in vigour-img, for example. Best way to achieve this would be to extract it to a separate repo. I'm just waiting for Marcus & Jim's approval of this "pattern" before going crazy and using it everywhere, in case there is a major drawback I haven't tought of.

shawninder commented 9 years ago

One this is done, a new issue in that repo should be to make "getter" obsolete and remove the "getter" fields from vigour-packer-server/config.js

shawninder commented 9 years ago

@jimdebeer @marcusbesjes Guys, any feedback on this? You can look at how it's made in https://github.com/vigour-io/vigour-packer-server/blob/master/config.js and https://github.com/vigour-io/vigour-packer-server/blob/master/bin/index.js

shawninder commented 9 years ago

This config system should support multiple commands, maybe using the same pattern as gaston

jimdebeer commented 9 years ago

both links are mysteriously 404's

shawninder commented 9 years ago

fixed

shawninder commented 9 years ago

But you'll need to be logged in as it's a private repo (otherwise you'll get 404 again)

shawninder commented 9 years ago

if you can see this issue you should be able to see the files

shawninder commented 9 years ago

Now exists as a separate repo: https://github.com/vigour-io/pliant Next step: have the packer use that repo and make temp.js obsolete