Skip to content
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: alias module so we can access process.server in vite node build #462

Merged
merged 4 commits into from
Apr 24, 2021

Conversation

danielroe
Copy link
Member

@danielroe danielroe commented Apr 23, 2021

  1. adds ~composition-api alias to allow vite plugins to run on module code (for process.server)
  2. inlines https://github.com/unjs/defu temporarily (until packaged with .mjs) - this only became an issue after step 1 above

cc: @pi0

with these changes nuxt/vite#127 should be mergeable

@danielroe danielroe requested a review from pi0 April 23, 2021 10:46
@pi0
Copy link
Member

pi0 commented Apr 23, 2021

@danielroe Can you please upgeade ufo to 4.0.1? A silly typo fixed :}

@danielroe danielroe merged commit 7c997c5 into main Apr 24, 2021
@danielroe danielroe deleted the fix/vite-issues branch April 24, 2021 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants