New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace babel-5 parser with retrocompatible babylon setup #230

Merged
merged 2 commits into from Dec 7, 2017

Conversation

Projects
None yet
4 participants
@slorber
Contributor

slorber commented Nov 24, 2017

See #229

We can get rid of Babel5 as the dependency on Babylon is enough to make the tests pass, as long as we use Babylon in the same way Babel5 used to use it under the hood

@slorber slorber changed the title from Replace babel-5 and use retrocompatible babel5 parsing with babylon to Replace babel-5 parser with retrocompatible babylon setup Nov 24, 2017

@sokra

This comment has been minimized.

sokra commented Dec 4, 2017

@fkling Could you take a look at this? I would really great to get rid of the security warning...

@cpojer cpojer merged commit 07ddf69 into facebook:master Dec 7, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment