Skip to content
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

Update repl to Babel 7 #1915

Merged
merged 3 commits into from Feb 7, 2019
Merged

Update repl to Babel 7 #1915

merged 3 commits into from Feb 7, 2019

Conversation

nicolo-ribaudo
Copy link
Member

@nicolo-ribaudo nicolo-ribaudo commented Dec 11, 2018

  • Remove separate handling of Babel and preset-env version, since
    they are both 7 now
  • Remove code for the "debug" option, which was already disabled
    for v7

(Hide whitespace changes when reviewing)

Fixes #1901

- Remove separate handling of Babel and preset-env version, since
  they are both 7 now
- Remove code for the "debug" option, which was already disabled
  for v7
@babel-bot
Copy link
Contributor

babel-bot commented Dec 11, 2018

Deploy preview for babel ready!

Built with commit f9d729b

https://deploy-preview-1915--babel.netlify.com

@@ -109,7 +109,7 @@ function compileCode(sourceEditor, targetEditor) {

try {
transformed = Babel.transform(sourceEditor.getValue(), {
presets: ["latest"],
presets: ["es2015", "es2016", "es2017"],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should would want to get preset-env to be default eventually right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't want to load two different bundles just for the minirepl.
I think that, since it is just a demo, it is ok like this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

totally didn't see it was the minirepl, makes sense!

Copy link
Member

@danez danez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants