Skip to content

fix(env): Remove section about having to use dotenv to use env vars in code - #4860

Closed
janpio wants to merge 3 commits into
mainfrom
janpio-patch-19
Closed

fix(env): Remove section about having to use dotenv to use env vars in code#4860
janpio wants to merge 3 commits into
mainfrom
janpio-patch-19

Conversation

@janpio

@janpio janpio commented Jun 7, 2023

Copy link
Copy Markdown
Contributor

.env file is automatically loaded, so no dotenv needed.

closes prisma/orm#19519

@vercel

vercel Bot commented Jun 7, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 15, 2023 2:31pm

@github-actions

github-actions Bot commented Jun 7, 2023

Copy link
Copy Markdown
Contributor

This PR changes the following pages (Vercel Preview Deploy links):

Changed Vercel links

(Note that links will only be valid after Vercel preview deploy succeeded)

@github-actions

github-actions Bot commented Jun 7, 2023

Copy link
Copy Markdown
Contributor

This PR probably requires the following redirects to be added to vercel.json:

  • This PR does not change any pages in a way that would require a redirect.

@millsp

millsp commented Jun 7, 2023

Copy link
Copy Markdown
Contributor

The removal is correct, but I am not sure this closes the mentioned issue? The person is complaining that Prisma Client loads and pollutes their env although it shouldn't (I think we have multiple reports of that?).

@janpio

janpio commented Jun 7, 2023

Copy link
Copy Markdown
Contributor Author

Yes, as you say we have other issues about that. This user was mostly misguided by the docs that tell them otherwise - so fixing the docs solves their immediate complaint.

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.

importing prisma client pollutes process.env without using dotenv

4 participants