-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Issue/1933 Create "Migrating" section and move content from Guides #1948
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
| --- | ||
|
|
||
| The following sections describe the major changes from webpack 1 to 2. | ||
|
|
||
| T> Note that there were far fewer changes between 2 and 3, so that migration shouldn't be too bad. If you are running into issues, please see [the changelog](https://github.com/webpack/webpack/releases) for details. | ||
|
|
||
| W> This content may be moved to the blog post in the near future as version 2 has been out for a while. On top of that, version 3 was recently released and version 4 is on the horizon. As noted above, folks should instead to refer to [the changelog](https://github.com/webpack/webpack/releases) for migrations. |
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 dont think this is relevant anymore
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.
Previous T> Note is also leading to https://github.com/webpack/webpack/releases
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.
Agreed
TheLarkInn
left a comment
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.
Looks great!
TheLarkInn
left a comment
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.
@EugeneHlushko we should merge this against master instead of next branch. Can you update this PR to reflect that?
7398857 to
3ae1c22
Compare
|
Thank you, it is now |
| --- | ||
|
|
||
| The following sections describe the major changes from webpack 1 to 2. | ||
|
|
||
| T> Note that there were far fewer changes between 2 and 3, so that migration shouldn't be too bad. If you are running into issues, please see [the changelog](https://github.com/webpack/webpack/releases) for details. | ||
|
|
||
| W> This content may be moved to the blog post in the near future as version 2 has been out for a while. On top of that, version 3 was recently released and version 4 is on the horizon. As noted above, folks should instead to refer to [the changelog](https://github.com/webpack/webpack/releases) for migrations. |
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.
Take a look at changes done in #1939.
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.
It will be fine, any of these getting into master first and the other gets rebased
|
@TheLarkInn can you re-approve this one been hanging for some time now |
|
Hi @montogeek , |
|
@EugeneHlushko sure |
43107b2 to
f5940e1
Compare
|
Failure is known: API rate limit exceeded for 104.154.120.187 Pls finish this one @montogeek |
|
@EugeneHlushko Do you have permission to move your branch to this repo? |
|
@montogeek yes, makes any sense for the finished PRs? thought to create new ones under main repository but merge older ones as is to reduce mess on my side of things |
|
Was trying to make the build pass, it already did. Pinged @TheLarkInn to approve it |
|
Thank you! |
376cad5 to
359cdf9
Compare
antwar.config.js
Outdated
| 'why-webpack': '/comparison', | ||
| 'production-build': '/guides/production' | ||
| 'production-build': '/guides/production', | ||
| 'migrating': '/migrate/3', |
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.
Indent
|
Approved. Let's get conflicts resolved and good to go! |
359cdf9 to
dbdb981
Compare
|
Cool, waiting for build and can merge cc @montogeek |
|
Thanks |
Create "Migrating" section and move content from Guides.
Fixes #1933