Skip to content
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

Replace fast-glob with fdir or tinyglobby #3307

Closed
1 task done
benmccann opened this issue Aug 26, 2024 · 2 comments
Closed
1 task done

Replace fast-glob with fdir or tinyglobby #3307

benmccann opened this issue Aug 26, 2024 · 2 comments
Labels

Comments

@benmccann
Copy link

Is there an existing issue that is already proposing this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe it

fast-glob has 17 dependencies and really isn't needed where it's used
https://npmgraph.js.org/?q=fast-glob

Describe the solution you'd like

Ideally replace with fdir since globbing isn't actually needed in this case
Alternatively, replace with tinyglobby, which has a more 1:1 API and would be a slightly faster migration

Teachability, documentation, adoption, migration strategy

No response

What is the motivation / use case for changing the behavior?

Reduce the number of unnecessary dependencies

@kamilmysliwiec
Copy link
Member

We plan to move to the built-in glob (sooner or later) nodejs/node#51912

@benmccann
Copy link
Author

yeah, I figured that will take at least a couple of years though until everyone's running a supported version of Node

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants