search

AvianFlu / ncp

Asynchronous recursive file copying with Node.js.
MIT License
680 stars 103 forks source link

Promise API #104

Closed RomkeVdMeulen closed 8 years ago

RomkeVdMeulen commented 8 years ago

I have a very involved buildscript, with nested buildsteps and callbacks several layers deep. I'd like to rewrite that as a chain of .then().

Are there any plans to support a Promise API with ncp?

edef1c commented 8 years ago

done: var ncp = Promise.denodeify(require('ncp').ncp) (if your promise implementation doesn't have .denodeify or an equivalent, maybe check out https://www.npmjs.com/package/promise)

mmalecki commented 8 years ago

@nathan7 got it right. We're not going to implement a Promise API in ncp, so closing this issue.