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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[core] Remove outdated Next.js options #33845
[core] Remove outdated Next.js options #33845
Conversation
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.
For the warning standing, it's a regression, fixed in vercel/next.js#39171.
ignoreBuildErrors: true, | ||
}, | ||
webpack5: true, |
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.
Webpack 4 was removed in Next.js v12 https://github.com/vercel/next.js/blob/37d11008250b3b87dfa4625cd228ac173d4d3563/errors/webpack5.md
@@ -33,10 +33,8 @@ module.exports = { | |||
}, | |||
typescript: { | |||
// Motivated by https://github.com/vercel/next.js/issues/7687 | |||
ignoreDevErrors: true, |
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.
Removed in vercel/next.js#12941
(I hate this kind of PRs, when you look at it, you have no idea why ignoreDevErrors
is gone. And of course, the PR is locked, so you can't ask, it sounds like it was decided on a private channel, perfect to engage with the community)
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.
Thanks for the fix!
I wanted to check the logs server side, and here is what I got, too much noise for my taste 馃榿:
It comes from #33626