Skip to content

Conversation

wfleming
Copy link
Contributor

This reverts commit df26467, reversing
changes made to 804d90f.

There are cases that cause the stack to blow up with Babel, apparently: eslint/eslint#3440

These issues seem to be fixed in eslint's master, but they're currently
in beta of a 2.0 release, so trying to merge in that with our changes
seems like a riskier change to make right now: we should wait until
they release 2.0 & take another look. For now, if you're using ES6, you
can opt into this config

cc @codeclimate/review

This reverts commit df26467, reversing
changes made to 804d90f.

There are cases that cause the stack to blow up with Babel, apparently: eslint/eslint#3440

These issues seem to be fixed in eslint's master, but they're currently
in beta of a 2.0 release, so trying to merge in that with our changes
seems like a riskier change to make right now: we should wait until
they release 2.0 & take another look. For now, if you're using ES6, you
can opt into this config
@gdiggs
Copy link
Contributor

gdiggs commented Jan 25, 2016

LGTM

wfleming added a commit that referenced this pull request Jan 25, 2016
Revert babel as the default parser.
@wfleming wfleming merged commit 0e6817c into master Jan 25, 2016
@wfleming wfleming deleted the will/revert-eslint branch January 25, 2016 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants