Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

System errors should have scope too #714

Closed
valueof opened this Issue · 0 comments

1 participant

@valueof
Owner

Right now we don't add any scope to system errors such as "Too many errors." one. This is inconsistent with all other error objects and should be fixed.

@valueof valueof was assigned
@valueof valueof closed this issue from a commit
@valueof valueof Add 'scope' property to critical errors.
Critical errors (those produced by the 'quit' function) show up
in JSHINT.data() as well as any other errors and warnings and
should have the same format.

This patch adds a missing property 'scope' to such errors. It's
always set to '(main)' since errors are unrecoverable.

Fixes GH-714.
ee77638
@valueof valueof closed this in ee77638
@jugglinmike jugglinmike referenced this issue from a commit in jugglinmike/jshint
@valueof valueof Add 'scope' property to critical errors.
Critical errors (those produced by the 'quit' function) show up
in JSHINT.data() as well as any other errors and warnings and
should have the same format.

This patch adds a missing property 'scope' to such errors. It's
always set to '(main)' since errors are unrecoverable.

Fixes GH-714.
5275ad7
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.