search

hoodiehq-archive / node-multicouch

Launch multiple CouchDBs from the same installation.
30 stars 18 forks source link

Facilitate node_modules/.bin/multicouch symlink #12

Closed natevw closed 10 years ago

natevw commented 10 years ago

This enables the CLI via the normal npm means. See https://npmjs.org/doc/json.html#bin

(Also revs package version to be ready for npm publish ;-)

svnlto commented 10 years ago

Bumping the version in a PR is sort of an anti pattern, please amend.

janl commented 10 years ago

the binary is for testing only and not meant for production use.

janl commented 10 years ago

(we could have a binary that is supposed to for production use, but it would have to be written from scratch, ideally in a separate package: multicouch-cli)

natevw commented 10 years ago

Ok, thanks @svnlto I've never been quite sure but makes sense. I'll leave it as-is for now since it sounds like Jan has other plans.

@janl — Could you elaborate a bit? Are you just wanting to make the Couch-running modules as granular as possible? If the library is for production use, and the CLI just wraps the library, what is the harm in "allowing" the CLI for production use?

janl commented 10 years ago

@natevw there is nothing wrong with allowing “a” cli to wrap the library and be good for production use. It’s just that today bin/multicouch is not that binary. And instead of rewriting or fixing that one, I suggested a separate package.