perf(rollup): skip URI-scheme ids before this.resolve()#4253
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
commit: |
|
Thanks. Pending as i have to carefully review for backport. |
🔗 Linked issue
spotted this while investigating nuxt/nuxt#27106.
related: nuxt/nuxt#34753
this makes a measureable difference in terms of nuxt build performance
❓ Type of change
📚 Description
this is a perf improvement porting some changes that have already been made in
mainbranch (but which I discovered somewhat differently via nuxt/nuxt#27106):node:builtinsdata:,http(s):,virtual:,bun:,deno:, etc.📝 Checklist