Skip to content

Add Babel.js compiler support. #312

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

Merged
merged 8 commits into from
Feb 9, 2016
Merged

Add Babel.js compiler support. #312

merged 8 commits into from
Feb 9, 2016

Conversation

nlutsenko
Copy link
Contributor

This finally unblocks us from using fancy new things like ES6 and friends.
Also, there was a looot of invalid JavaScript here and there, which Babel compilation enforces to be valid - that's fixed as well.

@nlutsenko
Copy link
Contributor Author

cc @drew-gross

@drew-gross
Copy link
Contributor

Seems solid to me, just two things: This probably warrants an update to CONTRIBUTING.md, and, is the removal of the commas after the last key of object due to babel? Those work fine in the dashboard, I think they should be fine here as well.

@nlutsenko
Copy link
Contributor Author

Yup, you are right.
Looks like I broke istanbul reports, will fix it and only then merge.

@facebook-github-bot
Copy link

@nlutsenko updated the pull request.

@nlutsenko
Copy link
Contributor Author

W00t, finally have everything running. Waiting for tests and merging then.

@nlutsenko nlutsenko assigned nlutsenko and unassigned gfosco Feb 9, 2016
@facebook-github-bot
Copy link

@nlutsenko updated the pull request.

nlutsenko added a commit that referenced this pull request Feb 9, 2016
@nlutsenko nlutsenko merged commit f2c33c0 into master Feb 9, 2016
@nlutsenko nlutsenko deleted the nlutsenko.babel branch February 9, 2016 06:08
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.

4 participants