-
Notifications
You must be signed in to change notification settings - Fork 52
Description
hi ππΌ,
it would be great if this repo could migrate from glob β tinyglobby.
the glob dependency is old: pinned to 10.x while upstream is at 13.x (see glob on npm), and recently had security issues, e.g. #822 which was not important for this project but affects many downstreams b/c npm audit raises alarm bells about it, etc.
glob has good modern replacements, specifically tinyglobby which is lean and fast (see e.g. https://e18e.dev/blog/tinyglobby-migration and e18e/ecosystem-issues#218), stable, popular, e.g. used by vite (which even indirectly exposes it in its API; see e.g. these docs).
there are only 3 very similar call sites in this code base, e.g.
await glob(globAssets, {
absolute: true,
nodir: true, // We need individual files for preparation
ignore: options.sourcemaps?.ignore,
})β¦ which all seem to have direct tinyglobby equivalents: absolute and ignore can be used as-is, and nodir β onlyFiles.
thanks in advance for your consideration!
Metadata
Metadata
Assignees
Labels
Projects
Status