-
Notifications
You must be signed in to change notification settings - Fork 185
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
Build error after upgrade to 6.12.2
from 6.12.1
#909
Comments
My ...
tailwindcss: {
cssPath: ['~/assets/css/tailwind.css', { injectPosition: 'last' }],
configPath: 'tailwind.config.ts',
},
... I found that everything goes well if I remove Maybe there are some breaking changes for this redundant config. |
Thanks for raising this. I see the repository on your profile, and I'll try to investigate further. You don't need to provide configPath if it is already tailwind.config (and maybe the path resolution behaves odd without path specifier eg relative or project dir root). Still, it shouldn't cause issues between versions. From the traceback, I'm thinking if it's this line - https://github.com/nuxt-modules/tailwindcss/blob/main/src/runtime/merger.js#L34 Again, this wasn't changed between 6.12.1 and 6.12.2 but I notice that the if condition rather belongs in the earlier block (how did we miss that 😄). Let me continue investigating |
Hey, @ineshbose any update on this? We've upgraded nuxt/ui which depends on @nuxtjs/tailwindcss and now we have to lock the version of tailwindcss via resolutions. |
It's possible that #913 could address this as it is changing config-loading strategy. Once merged, I shall be asking for feedback! |
Merged! Could you give https://tailwindcss.nuxtjs.org/getting-started/installation#opting-into-the-nightly-channel |
@ineshbose I ran into another error with the nightly version:
|
Environment
Reproduction
No response
Describe the bug
Build error after upgrade to
6.12.2
from6.12.1
Additional context
No response
Logs
No response
The text was updated successfully, but these errors were encountered: