Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Added the command to install babel-preset for babel-loader #8058

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 3 commits into from
Jul 11, 2017

Conversation

bogdaniel
Copy link
Contributor

While testing this bundle i was getting hit by an erorr that i can't find the babel preset es2017 which i solved it by installing.
I'm new to this webpack thing and to work like this with javascript. Some people might be in my position and try for hours to solve this. :D

While testing this bundle i was getting hit by an erorr that i can't find the babel preset es2017 which i solved it by installing.
I'm new to this webpack thing and to work like this with javascript. Some people might be in my position and try for hours to solve this. :D
Copy link
Member

@weaverryan weaverryan left a comment

Choose a reason for hiding this comment

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

Small comment, but I like it!

But first let's install the babel preset.

``yarn add babel-preset-es2017``
This will install the preset that we will use later to configure the babel.
Copy link
Member

Choose a reason for hiding this comment

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

I think this is indeed a good clarification! But what about adding this information as comments in the code block below (right above the es2017 line. We can say something like:

// make sure to install any presets you want to use
// yarn add ...

@bogdaniel
Copy link
Contributor Author

I hope it is ok now :).

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

I propose a minor reword to make the comment easier to understand. But if you don't agree with it, please revert. Thanks!

@bogdaniel
Copy link
Contributor Author

@javiereguiluz yeap it's ok =D

@xabbuh
Copy link
Member

xabbuh commented Jul 11, 2017

Thank you @bogdaniel.

@xabbuh xabbuh merged commit b9edef8 into symfony:3.3 Jul 11, 2017
xabbuh added a commit that referenced this pull request Jul 11, 2017
…r (bogdaniel, javiereguiluz)

This PR was merged into the 3.3 branch.

Discussion
----------

Added the command to install babel-preset for babel-loader

While testing this bundle i was getting hit by an erorr that i can't find the babel preset es2017 which i solved it by installing.
I'm new to this webpack thing and to work like this with javascript. Some people might be in my position and try for hours to solve this. :D

Commits
-------

b9edef8 Minor reword
ab41baf Update babel.rst
e74e556 Added the command to install babel-preset for babel-loader
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.

5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.