Skip to content

Replace globby/fast-glob/tiny-glob with tinyglobby#13299

Merged
ematipico merged 3 commits intomainfrom
replace-with-tinyglobby
Feb 24, 2025
Merged

Replace globby/fast-glob/tiny-glob with tinyglobby#13299
ematipico merged 3 commits intomainfrom
replace-with-tinyglobby

Conversation

@bluwy
Copy link
Member

@bluwy bluwy commented Feb 23, 2025

Changes

Had some time to finish up a PR I made in the past (#12165)

This PR replaces the globbing libraries to tinyglobby as it's smaller and used by other packages, such as Vite and the next version of changesets. I've also replaced micromatch to picomatch as that's used by tinyglobby.

Testing

The existing tests should pass

Docs

n/a

@github-actions github-actions bot added pkg: integration Related to any renderer integration (scope) pkg: astro Related to the core `astro` package (scope) labels Feb 23, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 23, 2025

CodSpeed Performance Report

Merging #13299 will not alter performance

Comparing replace-with-tinyglobby (3d30dd1) with main (baf0c40)

Summary

✅ 6 untouched benchmarks

@changeset-bot
Copy link

changeset-bot bot commented Feb 24, 2025

🦋 Changeset detected

Latest commit: 3d30dd1

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ematipico ematipico force-pushed the replace-with-tinyglobby branch from 6eda1be to 3d30dd1 Compare February 24, 2025 12:04
@ematipico ematipico merged commit 2e1321e into main Feb 24, 2025
17 checks passed
@ematipico ematipico deleted the replace-with-tinyglobby branch February 24, 2025 12:50
@astrobot-houston astrobot-houston mentioned this pull request Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope) pkg: integration Related to any renderer integration (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants