-
Notifications
You must be signed in to change notification settings - Fork 977
feat(unplugin): add scanPackages option
#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
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. |
scanPackages option
benjamincanac
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've renamed to scanPackages and updated the docs.
π Linked issue
Resolves #4892
β Type of change
π Description
Allows adding external packages to be scanned for Nuxt UI components through the new
scanPackagesoption. This allows using Nuxt UI as a dependency for other projects.π Checklist