Skip to content

Fix some grammatical errors in 5.mdx #5290

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.

Sign up for GitHub

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 1 commit into from
Aug 14, 2021
Merged

Fix some grammatical errors in 5.mdx #5290

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions src/content/migrate/5.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ contributors:
- rramaa
- chenxsan
- jamesgeorge007
- getsnoopy
---

This guide aims to help you migrating to webpack 5 when using webpack directly. If you are using a higher level tool to run webpack, please refer to the tool for migration instructions.
Expand Down Expand Up @@ -179,9 +180,9 @@ console.log(pkg.version);
- This doesn't apply to the `webpack(..., callback)` form which automatically closes.
- This is optional if you use webpack in watching mode until the user ends the process. The idle phases in watch mode will be used for this kind of work.

### Run a single build and follow advises
### Run a single build and follow advice

Please make sure to read the building errors/warnings carefully. If there is no corresponding advise, please create an issue and we will try to resolve it.
Please make sure to read the building errors/warnings carefully. If there is no corresponding advice, please create an issue and we will try to resolve it.

Repeat the following steps until you solved at least level 3 or 4:

Expand All @@ -203,7 +204,7 @@ Repeat the following steps until you solved at least level 3 or 4:

- Level 5: **Runtime Errors**.

This is tricky. You probably have to debug to find the problem. A general advise is difficult here. But we do list some common advices below regarding Runtime Errors:
This is tricky. You probably have to debug to find the problem. General advice is difficult here. But we do list some common advice below regarding Runtime Errors:

- `process` is not defined.
- webpack 5 does no longer include a polyfill for this Node.js variable. Avoid using it in the frontend code.
Expand All @@ -221,7 +222,7 @@ Repeat the following steps until you solved at least level 3 or 4:
You probably get a lot of deprecation warnings. This is not directly a problem. Plugins need time to catch up with core changes. Please report these deprecations to the plugins. These deprecations are only warnings and the build will still work with only minor drawbacks (like less performance).

- You can hide deprecation warnings by running node with `--no-deprecation` flag, e.g.: `node --no-deprecation node_modules/webpack/bin/webpack.js`. This should only be a temporary workaround.
- Plugins and Loaders contributors can follow the advises in the deprecation messages to improve the code.
- Plugins and Loaders contributors can follow the advice in the deprecation messages to improve the code.

- Level 7: **Performance issues**.

Expand Down