-
Notifications
You must be signed in to change notification settings - Fork 26.4k
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
Next.js 9.2 not backward compatible with postcss config #10130
Comments
Duplicate of #10117. Keeping it open there as it already has an explanation etc.
It's not a breaking change, you were using |
@timneutkens Oh, I didn't realize this was an issue with |
This issue has been automatically locked due to no recent activity. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you. |
I don't really know how to classify this. Maybe just something that should be documented.
I have a
postcss.config.js
file that returns a function (i.e takesctx
as a parameter, returns a config object). That works fine in Next.js 9.1. But in 9.2, I get this error:Looking at the code for
plugin.ts
, this is obviously intentional behavior. I tried to find this breaking change documented somewhere (e.g. release notes, blog posts, etc.), but didn't find anything (the pull request with the change does provide some context, but it's hard to find). I get it, new version, things change. So possibly the only resolution to this is to document it somewhere in the release notes, maybe explain why it's changed.If it's helpful, this is what my postcss.config.js file looked like before. Granted, what I'm using
ctx
for is probably unnecessary (sourcemaps seem to be generated just fine using Next.js default behavior). So it's not a big deal for me to get rid of it.System information
The text was updated successfully, but these errors were encountered: