-
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
feat: extends support #499
Conversation
Agree for an edge release to beta test this! |
I do not think extending external Extending support is also a good idea but please note that the current implementation is using undocumented |
Co-authored-by: pooya parsa <[email protected]>
Co-authored-by: Sébastien Chopin <[email protected]>
@atinux ; published |
I believe this one is still an improvement in term of DX for theme author since a tailwind.config is needed for Intelissense to work. |
This did not resolve #500, which is still an open issue on 5.3.0 |
This part has been moved into nuxt/content#1351
The PR now resolves #498 and #488 by providing support for
extends
layers in the configPath resolving.It should also resolve #500, that has been introduced by
5.2.0
.configPath
key now supports:configPath: 'my-config/tailwind.config'
configPath: ['my-configs/tailwind.config', ...]
These ultimately gets resolved and merged layer-by-layer using
defu
.This PR updates the structure of the
module.ts
file to allow such resolving, so should be reviewed with care.I checked it myself and can't see any breaking changes, but I would appreciate anyone with some time to review it.
Maybe we could consider making an
edge
release for this branch @atinux ?