-
-
Notifications
You must be signed in to change notification settings - Fork 6.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
docs: update pre-bundling guide #7008
Conversation
I have a question on how to handle hmr on optimized deps? And there is another dev situation, I have some pnpm linked packages by pnpm link --glboal xxxx, not located in monorepo packages, the linked pkg is umd format and need to be transpiled to esm format, how can I use the linked developing pkg for debugging? This pkg works well by installing it from the registry instead linking from pnpm-global folder. |
HMR with linked optimized deps isn't great right now as you have to re-prebundle with the Re pnpm link with |
I don't think this fixed #2697 like you claim. It doesn't support the linked mono-repo syntax or explain how to use that. It's not working for me. |
@ScubaDaniel It doesn't fix it, but closes it as the documentation highlights the changes needed to support it. Perhaps it's something we can revisit with the recent change to the optimizer, but for now I'm not sure what's not working for you. It would be great if you can outline it in a GitHub discussion, or in the discord server. |
I've been struggling with this problem now for days, going round in circles and I think I've just figured out why. Looking at the in browser source files (in chrome) I can see that only my I've tried and Is there something obvious I've missed? Or was there a decision made that packages must export everything from the top level? |
Description
Closes #3898
Closes #5566
Closes #5668 (@aleclarson do you have a separate solution to this?)
Closes #2928
Closes #2697
npm pack
mention #6982 is incorrect)Additional context
Was building a plugin to automate the tedious "restart dev server with
--force
" part, but was stuck with #7004. I could probably useoptimizeDeps
to workaround it, but that'll be an endeavour for another day.What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).