Skip to content
This repository was archived by the owner on Apr 6, 2023. It is now read-only.
/frameworkPublic archive

fix(nuxt): replace process.dev in nitro bundle - #5852

Merged
pi0 merged 3 commits into
mainfrom
fix/vite-node-dev
Jul 12, 2022
Merged

fix(nuxt): replace process.dev in nitro bundle#5852
pi0 merged 3 commits into
mainfrom
fix/vite-node-dev

Conversation

@antfu

@antfuantfu commented Jul 12, 2022

Copy link
Copy Markdown
Member

process.dev is undefined in the npm build (while it works in the local playground). Since NUXT_VITE_NODE_OPTIONS will only be injected in dev, I guess it's safe to remove it

@netlify

netlifyBot commented Jul 12, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs canceled.

NameLink
🔨 Latest commit245a39b
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/62cd688992732b000976f354

@antfuantfu changed the title fix(vite-node): process.dev undefined in production causing FOUCfix(vite-node): process.dev undefined in npm release causing FOUCJul 12, 2022
@antfuantfu changed the title fix(vite-node): process.dev undefined in npm release causing FOUCfix(core): replace process.dev in NitroJul 12, 2022
@pi0pi0 changed the title fix(core): replace process.dev in Nitrofix(nuxt): replace process.dev in nitro bundleJul 12, 2022
pi0
pi0 approved these changes Jul 12, 2022
@pi0
pi0 merged commit 135dfdf into mainJul 12, 2022
@pi0
pi0 deleted the fix/vite-node-dev branch July 12, 2022 12:32
@pi0pi0 mentioned this pull request Jul 12, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@antfu@pi0@danielroe