-
Notifications
You must be signed in to change notification settings - Fork 325
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
fix: automatic dev mode detection #1106
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
🦋 Changeset detectedLatest commit: 64ddb63 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
WalkthroughThis pull request involves modifications to the Changes
Possibly related PRs
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
📦 Bundle size comparison
|
A new canary is available for testing. You can install this latest build in your project with: pnpm add @uploadthing/expo@7.1.5-canary.8479c92
pnpm add @uploadthing/mime-types@0.3.4-canary.8479c92
pnpm add @uploadthing/nuxt@7.1.5-canary.8479c92
pnpm add @uploadthing/react@7.1.5-canary.8479c92
pnpm add @uploadthing/shared@7.1.5-canary.8479c92
pnpm add @uploadthing/solid@7.1.5-canary.8479c92
pnpm add @uploadthing/svelte@7.1.5-canary.8479c92
pnpm add uploadthing@7.4.3-canary.8479c92
pnpm add @uploadthing/vue@7.1.5-canary.8479c92 |
this bug is so cursed...
NODE_ENV is set when the docs are built on Vercel, which causes the packages to be built with this variable defined.
and then when we go and publish pacakges we're pickign up that cached build and getting the
process.env.NODE_ENV
statically replaced...tried fixing this by omitting the variable from the turbo process in #1105 but that was pure pain..
fix by patching bunchee to not append NODE_ENV to the
@rollup/plugin-replace
pluginSummary by CodeRabbit
Chores
bunchee@6.1.2
to resolve dependency configurationBug Fixes
uploadthing
module