-
-
Notifications
You must be signed in to change notification settings - Fork 778
docs: add code splitting section #3991
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
|
@onmax is attempting to deploy a commit to the Nitro Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughTwo documentation pages were updated to add "Code splitting" explanations describing per-route-handler chunking and on-demand loading, and to expand the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
docs/1.docs/5.routing.md
Outdated
|
|
||
| ```ts | ||
| export default defineNitroConfig({ | ||
| inlineDynamicImports: true |
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.
We could just link it to configuration section for changing behavior. Some deployment presets are forced to do this (not as benefit really, it increases cold start actually!), not in any normal conditions it is good idea to enable flag.
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.
Addressed in ef3e460
pi0
left a comment
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.
thnx!
Code splitting is a key Nitro feature but wasn't documented. Added section explaining how route handlers are chunked and how to disable it with
inlineDynamicImports.