-
Notifications
You must be signed in to change notification settings - Fork 970
feat(unplugin): add option to allow external packages to scan for components #5510
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
base: v4
Are you sure you want to change the base?
Conversation
|
I wasn't sure about the name or location for the option. Besides the current option, I also considered extending |
commit: |
|
I'm wondering if it wouldn't be easier to just remove this exclude, do you have a reproduction to try if it doesn't decrease performances? |
Wouldn't completely removing the exclude mean all |
|
@romhml What do you think of this? Maybe we should remove the hard-coded |
|
@benjamincanac Removing the @ISOR3X how do you plan to leverage this in your projects? Edit Nuxt UI's plugin config inside yours or ask users to configure it themselves? |
The former. My main reason for this PR was mainly because we use Nuxt UI at the company I work at. Our UI library pretty much just wraps Nuxt UI and applies some different defaults and adds some extra components. |
π Linked issue
Resolves #4892
β Type of change
π Description
Allows adding external packages to be scanned for Nuxt UI components through the new
allowedPackagesoption. This allows using Nuxt UI as a dependency for other projects.π Checklist