-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
cleanup(core): move esbuild to use tinyglobby #29453
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
View your CI Pipeline Execution ↗ for commit 97a5868.
☁️ Nx Cloud last updated this comment at |
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.
Please add the eslint entry as usual to avoid regressions
An alternative to nrwl#28037, switches the esbuild package to use `tinyglobby` instead of `fast-glob`. This improves performance a fair chunk and reduces install complexity/size.
@JamesHenry this should be all good now |
An alternative to #28037, switches the esbuild package to use
tinyglobby
instead offast-glob
.This improves performance a fair chunk and reduces install complexity/size.
cc @benmccann @JamesHenry