Removed last comma on object return, solves crash on IE9

[urodoz]

No description provided.

[benvinegar]

Thanks!

[mattrobenolt]

@benvinegar shouldn't uglify remove this when compiled?

[mattrobenolt]

And/or can we get our linter to handle this?

[benvinegar]

Re Uglify – not sure. But I think it should work with/without minification, IMHO.

And/or can we get our linter to handle this?

It appears the laxcomma JSHint rule was deprecated:

http://jshint.com/docs/options/#laxcomma

I'm in favor of switching to ESLint, as we've done with the main Sentry project.

[mattrobenolt]

But I think it should work with/without minification, IMHO.

Agreed, was just curious if this issue existed in the minified version, since that is the primary consumer.

I'm in favor of switching to ESLint, as we've done with the main Sentry project.

#386 👍