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 tinyglobby #131

Closed
benmccann opened this issue Sep 12, 2024 · 7 comments · Fixed by #132
Closed

Replace fast-glob with tinyglobby #131

benmccann opened this issue Sep 12, 2024 · 7 comments · Fixed by #132

Comments

@benmccann
Copy link

tiny-globby is a drop-in replacement for fast-glob and is much smaller

@metonym
Copy link
Collaborator

metonym commented Sep 12, 2024

I'm all for this.

Do you mean tiny-glob? Cannot find tiny-globby on NPM.

@benmccann
Copy link
Author

Oops. I got it right in the issue title, but not description. It's https://www.npmjs.com/package/tinyglobby

@metonym
Copy link
Collaborator

metonym commented Sep 12, 2024

Released in v0.20.1.


Thanks for flagging this, and for your broader work in making the ecosystem faster and safer.

@benmccann
Copy link
Author

Wow, thanks for the super fast turnaround. That's awesome

Thanks for flagging this, and for your broader work in making the ecosystem faster and safer.

Thanks for the kind words. I didn't notice this until you posted that link just now, but if this repo upgrades its dependency on svelte-preprocess that will slim things down for you as well. A couple of other dependencies like rollup look like they could be bumped as well which would help with dependency deduping

@metonym
Copy link
Collaborator

metonym commented Sep 12, 2024

Tracking in #134

@metonym
Copy link
Collaborator

metonym commented Sep 13, 2024

Commenting for posterity.

The quick wins:

  • Replacing fast-glob with tinyglobby
  • Upgrading svelte-preprocess from v5 to v6

...have resulted in:

  • 37 fewer transitive dependencies
  • Reduction of 3 deprecated modules to 0

Before ([email protected])

Screenshot 2024-09-13 at 8 50 56 AM

After ([email protected])

Screenshot 2024-09-13 at 8 51 00 AM

@benmccann
Copy link
Author

Woohoo! Thank you for this!!

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

Successfully merging a pull request may close this issue.

2 participants