-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Conversation
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
There was a problem hiding this 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!
frontend/encore/babel.rst
Outdated
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. |
There was a problem hiding this comment.
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 ...
I hope it is ok now :). |
There was a problem hiding this 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!
@javiereguiluz yeap it's ok =D |
Thank you @bogdaniel. |
…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
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