Skip to content

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.

5 participants