-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
feat(remix-dev): use native pnp resolution in esbuild 0.15+ #4154
Conversation
🦋 Changeset detectedLatest commit: e715c19 The changes in this PR will be included in the next version bump. This PR includes changesets to release 16 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
i was hoping for this PR 🤩 - i know nothing about pnp though, so I'll defer to @pcattori - but if it's good, perhaps we can bump up to 0.15.9 to also include the jsx name collision fixes :) |
@redabacha Could you please rebase your branch onto latest |
@mcansh : let's do some solid testing on the |
100% |
I just upgraded my team's very small internal app to this P/R and no issues thus far. I'll update if we run into anything. |
…pnp-resolution
…pnp-resolution
…pnp-resolution
This have been implemented in #4301 |
Closes: #
Testing Strategy: