-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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: do not override process.env #12227
Conversation
🦋 Changeset detectedLatest commit: 1a580b4 The changes in this PR will be included in the next version bump. 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 |
!snapshot env-override |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
Co-authored-by: Bjorn Lu <[email protected]>
!snapshot env-override |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
@bluwy can you have a look at the linked adapters PR? Fetching hangs but idk why (or should I ask Alex?) |
!snapshot env-override |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
I have upgraded astro and after this change my node adapter does not read any env variables anymore
I am running my server like this:
|
I faced this issue this morning, gonna investigate today |
seems to only happen if you have a |
Let's track in #12324 |
Changes
Testing
Should still pass. Adapter test PR at withastro/adapters#422 (it works)
Docs
N/A