-
Notifications
You must be signed in to change notification settings - Fork 26.1k
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
Remove lodash from external packages list #63601
Remove lodash from external packages list #63601
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @timneutkens and the rest of your teammates on Graphite |
Failing test suitesCommit: 124b8b0
Expand output● app-dir action handling › should handle actions correctly after navigation / redirection events
Read more about building and testing Next.js in contributing.md.
Expand output● app fetch build cache › should treat lodash as an external package
Read more about building and testing Next.js in contributing.md. |
Stats from current PRDefault BuildGeneral
Client Bundles (main, webpack)
Legacy Client Bundles (polyfills)
Client Pages
Client Build Manifests
Rendered Page Sizes
Edge SSR bundle Size
Middleware size
Next Runtimes
build cache
Diff detailsDiff for middleware.jsDiff too large to display |
What?
Since the work that Shu did for optimizing large barrel dependencies marking lodash as external is no longer required.
Closes NEXT-2899