-
Notifications
You must be signed in to change notification settings - Fork 76
Description
Hi,
I've wrapped this tool in my CI flow, and so far we're communicating with it through child_process.exec, which is combersome a little, but works.
I was trying to rewrite the part, and was hoping to do something of the spirit of:
var couchapp = require('couchapp');
async.waterfall(
[ couchapp.createApp.bind( pathToFile )
, function(app, next) {
async.parallel( arrDbUrls, app.push, next)
}
]
, done
)
but was surprised to see that it's not possible! the url is provided already to the couchapp.createApp - which means that if I want to push it to N integration envs I have to createApp N times....
I would like to check with you if I can work on a pull request that will not change the CLI interface, but will make this difference for API on the main module.
Ah,
I also learnt that the tool throws errors instead of passing them to callbacks... (tsk, tsk, tsk 😞) so I can fix that too in the same PR, if that's OK with you 😄
Thanks,
O