Skip to content

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented Jun 9, 2025

Ensure Webpack doesn't crash on linter warnings during development, similar to how linter errors were handled in PR #2440, commit a7f9c45.

Ensure Webpack doesn't crash on linter warnings during development,
similar to how linter errors were handled in PR #2440, commit a7f9c45.
@dgdavid
Copy link
Contributor Author

dgdavid commented Jun 9, 2025

I faced

<e> [webpack-dev-middleware] HookWebpackError: [eslint] 
<e> /path/to/project/src/components/core/SelectableDataTable.tsx
<e>   330:5  warning  Expected property shorthand  object-shorthand
<e> 
<e> ✖ 2 problems (1 error, 1 warning)
<e>   0 errors and 1 warning potentially fixable with the `--fix` option.
<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:157:11
<e> -- inner error --
<e> 
<e> caused by plugins in Compilation.hooks.processAssets
<e> 

when writing something like,

const sharedData = {
  columns: columns 
}

@dgdavid dgdavid requested a review from imobachgs June 9, 2025 16:20
@coveralls
Copy link

Coverage Status

coverage: 62.578%. remained the same
when pulling 755be10 on adjust-webpack-config
into 7d4339c on master.

@dgdavid dgdavid merged commit ad38649 into master Jun 9, 2025
9 checks passed
@dgdavid dgdavid deleted the adjust-webpack-config branch June 9, 2025 18:15
@imobachgs imobachgs mentioned this pull request Jun 30, 2025
imobachgs added a commit that referenced this pull request Jun 30, 2025
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.

4 participants