search

AvianFlu / ncp

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

Politely recommend CPR over NCP #100

Open stefanpenner opened 9 years ago

stefanpenner commented 9 years ago

I can appreciate the existing maintainers are likely busy, so in the interim I would like to recommend users the similar node_module CPR.

Milage may vary, but for my use-cases it was a drop-in replacement with the added benefit of nearly eliminating all my existing intermittent CI failures.

The maintainers appear active and responsive.

If i myself had more bandwidth I would offer my time to help maintain this project, unfortunately my queue is full and I hope to save others time and energy by making this recommendation.

I hope this doesn't cause any anger or hurt feelings. It absolutely is not my intent.

some relevant issues:

vinhtq commented 8 years ago

Thank you! Currently NCP has problem with filter! I tried to use CPR and everything works great!

lucidogen commented 8 years ago

Thanks. Please note that if you get uv_pipe errors, you need to use cpr in the main process (electron application).

kranthitech commented 8 years ago

Hi. NCP seems to have a transform function which allows me to change the content of the files before copying. Does CPR have such a functionality?. Thanks.

fresheneesz commented 8 years ago

I dropped in CPR in place of NCP and it worked without much code changing at all. I was having issues with NCP when updating from node v0.10 to v4, and this seems not to have the same problem.

hollowdoor commented 6 years ago

Also this issue https://github.com/AvianFlu/ncp/issues/98

trusktr commented 1 year ago

Filtering by exclusion (cpr) instead of inclusion (ncp) makes things easier, especially because negating regexes with negative lookahead in JS is difficult.