Don't use console.error() please

tcr / scissors

PDF manipulation in Node.js! Split, join, crop, read, extract, boil, mash, stick them in a stew.

Apache License 2.0

285 stars 45 forks source link

Don't use console.error() please #2

Closed yetzt closed 7 years ago

yetzt commented 11 years ago

I really like your module, but please throw errors and don't use console.error().

yetzt commented 10 years ago

well, this does not seem to be maintained any more. :(

wmbutler commented 10 years ago

Yeah, sort of a bummer. Really wanted to use this but there is no way to know a) how many pages are in a document and b) how to stop execution on error.

rikkertkoppes commented 9 years ago

Fixed on this temp fork: https://github.com/theapsgroup/scissors/

tedeh commented 9 years ago

It would be quite easy to have this library inherit from events.EventEmitter and emit error events instead.

cboulanger commented 7 years ago

Should be fixed as @rikkertkoppes's fork has been merged. For anything left, please open a PR.