-
-
Notifications
You must be signed in to change notification settings - Fork 1k
fix: remove postcss.config
from schema
#9181
fix: remove postcss.config
from schema
#9181
Conversation
Β Open in CodeSandbox Web Editor | VS Code | VS Code Insiders |
I brought up the reason for this PR on the Nuxt Discord: https://discord.com/channels/473401852243869706/1043174741135069184 Essentially, the docs seemed to indicate that this was a boolean, even though it seems like it should have been a string (path to a postcss.config.js). nuxt/nuxt#14121 seems to indicate this would be a moot point anyway if it is to be deprecated. Perhaps we can have a |
We don't want to support an external |
Understood. That makes sense in context of Nuxt modules. We may want to be explicit in the Nuxt docs somewhere then that Nuxt can only configure postcss via the nuxt.config - since postcss's docs explicitly say to include a postcss.config as the recommended method of configuring it. |
Thanks for your pull request @nathanchase β€οΈ Webpack and Schema were left over i forgot to finally include in 3.0.0.
Indeed it is a tricky situation when comes to external config files. Technically it is not a limitation. We can load postcss config and merge it with Nuxt config and also allow modules extend the merged result but more a consistency aspect since we don't also load external
Seems good idea. We can clarify it with JSDocs for configuration that we don't load it. But if you have an idea of somewhere more accessible that we talk about generic Nuxt and External config files that would be awesome ππΌ |
postcss.config
from schema
I'd say https://nuxt.com/docs/getting-started/configuration seems like a good candidate for where to describe how Nuxt does or doesn't support external config files. I'd say for sure to include the ones you mentioned - As you said, it is confusing since there are configuration files like I'll work on another PR and try to come up with a list of common config files that you might have in a Nuxt 3 project, and perhaps we can just create a side-by-side table saying how those configurations are integrated with Nuxt. |
PR added: #9189 |
Isn't this a breaking change? |
Yes, this is a breaking change. And technically vite still has the ability to use an external @pi0 shall we instead deprecate this config and then remove it entirely in 3.1? |
π Linked issue
resolves nuxt/nuxt#14121
β Type of change
π Description
π Checklist