Skip to content

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented May 12, 2025

This reverts commit bf0b1a2, which performed an npm update on April 30th, 2025 as part of #2315.

After such an upgrade, the Webpack development server became unresponsive after facing some linter-related errors. Attempts to resolve the issue through further dependency upgrades were unsuccessful. This revert restores the previous stable state until next reliable upgrade.

@dgdavid dgdavid requested a review from ancorgs May 12, 2025 13:15
Copy link
Contributor

@ancorgs ancorgs left a comment

Choose a reason for hiding this comment

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

I tried this and it solved the issue to me.

Thanks a lot, it was killing my development process.

@dgdavid dgdavid merged commit 94a945c into master May 13, 2025
8 checks passed
@dgdavid dgdavid deleted the revert-deps-upgrade branch May 13, 2025 09:35
@imobachgs imobachgs mentioned this pull request May 26, 2025
imobachgs added a commit that referenced this pull request May 26, 2025
dgdavid added a commit that referenced this pull request Jun 4, 2025
Ensure the dev server continues running after ESLint errors during
compilation, avoiding the need for manual resets.

Prior to this change, the server hangs after an ESLint update due to
errors like:

```
<e> ✖ 1 problem (1 error, 0 warnings)
<e>
<e>     at makeWebpackError (/path/to/project/node_modules/webpack/lib/HookWebpackError.js:48:9)
<e>     at /path/to/project/node_modules/webpack/lib/Compilation.js:3364:12
<e>     at eval (eval at create (/path/to/project/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:31:1)
<e>     at /path/to/project/node_modules/webpack/lib/Compilation.js:594:27
<e>     at /path/to/project/node_modules/eslint-webpack-plugin/dist/index.js:159:11
<e> -- inner error --
<e>
<e> caused by plugins in Compilation.hooks.processAssets
```

See webpack/eslint-webpack-plugin#280
and #2353.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants