-
Notifications
You must be signed in to change notification settings - Fork 26.6k
refactor: replace fast-glob
usage with tinyglobby
#60264
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
Conversation
368142c
to
c4899ce
Compare
Deployed adev-preview for 3b566a9 to: https://ng-dev-previews-fw--pr-angular-angular-60264-adev-prev-2w93toze.web.app Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt. |
Usage of the `fast-glob` package has been replaced with the `tinyglobby` package. The change reduces the number of transitive dependencies related to these packages from 17 to 2 while also maintaining equivalent functionality. This was also changed in the Angular CLI packages.
c4899ce
to
0d2f75c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! was thinking about this for the new change, but didn't want to introduce a new dep
caretaker note: please ignore the lint fail is it's due to the usual license checks. |
This PR was merged into the repository by commit 45f899a. The changes were merged into the following branches: main |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Usage of the
fast-glob
package has been replaced with thetinyglobby
package. The change reduces the number of transitive dependencies related to these packages from 17 to 2 while also maintaining equivalent functionality. This was also changed in the Angular CLI packages.