estools / escope

Escope: ECMAScript scope analyzer
BSD 2-Clause "Simplified" License
546 stars 77 forks source link

Drop using require'd assert in favour of just throwing an error #115

Open mrennie opened 7 years ago

mrennie commented 7 years ago

In a few places in the code, assert is require'd and used to throw a simple error.

You should remove this and simply 'throw new Error(...)'. The benefit of doing so is that it shrinks the browserify'd library considerably.