-
-
Couldn't load subscription status.
- Fork 23
feat: add new compileGlobs helper function
#168
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: main
Are you sure you want to change the base?
Conversation
|
I need to add some more logic since we cannot use picomatch directly... |
commit: |
|
I also need to add some tests (?). Here PR sent to unplugin-vue-components unplugin/unplugin-vue-components#885 : a working test using tinyglobby from local tgz from this PR can be found here https://github.com/unplugin/unplugin-vue-components/pull/885/files#diff-27fde5a56873e061c181b82af6cb092f8134719ba3eef739bfec2a3adc6f5bc8R27 (check also the test here https://github.com/unplugin/unplugin-vue-components/pull/885/files#diff-1df2465f0c401374a25cd07bf46305c86a09f87c96621c3a65660cddf226a523R36-R51 and the snap here https://github.com/unplugin/unplugin-vue-components/pull/885/files#diff-9780bc82cdbec33c105d9201f20c5c09342355082701a080dc07a890cbfb4e8fR33-R65). (ohhhh, pkg-pr-new there, I'm going to use it at unplugin-vue-components ✔️ done) |
|
We should split some code to allow reuse it here, maybe with changes here #170 |
|
@SuperchupuDev any hint if this will be added? I'm asking because I need to fix some repositories |
|
i haven't considered yet whether something like this will be added, plan is to get #170 merged first to avoid merge conflicts. if this wasn't added, would you have any other alternatives that solve your use case? |
|
Other than copy/pasting the code here and adding the dependency to unplugin vue components? No 😔 Just read the jsdocs, last paragraph. |
|
Another solution here is exporting the helper functions and so we can use them. |

I haven't found the docs repo, included a jsdoc to the new
compileGlobshelper function and so we can just move the content to the docs.Right now the jsdoc using
/**but should use/**/*.jsin the globs, adding the*.jsbreaks the module.resolves #166