53seven/node-blindparser

error strings

Hanfer opened this issue · 3 comments

it seems like it's most common to pass a node error object (e.g. new Error('whatever') ) as first parameter into the callback in case something goes wrong.
This module uses mostly plain strings which is rather confusing if one is used to the common way of doing it since if(err) return console.error(err.stack) results in the output undefined
if @dropdownmenu agrees i'd be happy to supply a pull request

Ill go through and clean that up. This was the first package that I wrote, so I didn't really conform to best practices yet

and thanks again :D