Skip to content

Use `errs` for Error creation and hoisting #184

Open
indexzero opened this Issue Mar 2, 2012 · 6 comments

7 participants

@indexzero
nodejitsu member

I think this would solve a lot of problems in jitsu.showError

@dscape
nodejitsu member
dscape commented Mar 2, 2012

+1

Btw @indexzero there's unmerged pull requests in err

@Marak
Marak commented Mar 21, 2012

Can someone elaborate on this ticket? We need additional information for someone to start working on this.

@gradus
gradus commented Apr 3, 2012

Yes. Please elaborate and I would be happy to help address this issue.

@Marak
Marak commented Apr 25, 2012

I think I now see some of the issues with jitsu.showError and where error handling is lacking.

Going to try and start catching and tagging certain errors before they hit this handler. Hopefully we can remove the majority fo the code in jitsu.showError soon.

@coleGillespie

What exactly are the issue you are seeing with jitsu.showError ?

@jfhbrook

The issue wish jitsu.showError is that it's a multi-hundred line error handling function. See: #321

I consider this issue (we have an errs library but aren't using it) to be a separate, and more minor issue.

@julianduque julianduque was assigned Mar 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.