-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: single env file #1092
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
feat: single env file #1092
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
🦋 Changeset detectedLatest commit: 8bb1294 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 |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://create-t3-app-git-julius-env-t3-oss.vercel.app/ |
Beside my two minor comments, I am sure that we could additionally use the type overwrite of |
I dont like the process.env typing since you miss stuff like default values for stuff your host provides, transforms/preprocess and other powerful primitives Zod provides besides the validation |
add prop to error msg
Which use cases do we need to test to make sure that this works as desired?
|
Hey t3-oss/translators! This PR contains changes to your language. Please review the changes ❤️. @a4addel @Sboonny @Defranos @joachimjusth @estubmo @josephayman @claudiofreitas @minsk-dev @Sn0wye @victoriaquasar @AmadeusTwi @ronanru @JohnBakhmat @fernandoxu @lodisy |
Co-authored-by: Muhammed Mustafa <[email protected]>
Good on my side. Ship? @c-ehrlich |
yea good for me also |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* single env file * docs * fixy * better comments * serverside -> server-side add prop to error msg * dont crash when calling next-auth in gssp * add min(1) to all string schema * Update www/src/pages/ar/usage/env-variables.md Co-authored-by: Muhammed Mustafa <[email protected]> * simplify error formatting * allow empty strings for discord env vars * incorrect import * here also * add changeset * don't leak env keys in prod * fix bad merge * cleanup --------- Co-authored-by: Christopher Ehrlich <[email protected]> Co-authored-by: Muhammed Mustafa <[email protected]>
Closes among others #1083 #1069 #1021
cc @iduuck
✅ Checklist
Changelog
Change to use a single env file as entry point. No need to import from server/client - we use a proxy to make sure the right errors are thrown if server vars are accessed on the client.
Screenshots
[Screenshots]
For example, if you try accessing
env.DATABASE_URL
on the client: