Raven 2.1.0 broke IE8

[brunis]

This line:
/(Raven|TraceKit)./.test(normalized['function']) ||
Was changed to:
/(Raven|TraceKit)./.test(normalized.function) ||

and broke IE8 completely

[mattrobenolt]

Hmm, so this was changed as a part of the switch to eslint.

I just ran changed this back to normalized['function'] locally, and eslint now yells at me.

/Users/matt/code/getsentry/raven-js/src/raven.js
  794:39  error  ["function"] is better written in dot notation  dot-notation

@benvinegar thoughts? Or does eslint just not care about IE8?

[mattrobenolt]

Looks like there's this allowedKeywords that defaults to true. :( http://eslint.org/docs/rules/dot-notation#allowkeywords