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: conditionally run envVars installer #196

Merged
merged 3 commits into from
Jul 15, 2022
Merged

fix: conditionally run envVars installer #196

merged 3 commits into from
Jul 15, 2022

Conversation

juliusmarminge
Copy link
Member

Only run the env-installer if choosing prisma or next-auth

  • I reviewed linter warnings + errors, resolved formatting, types and other issues related to my work
  • The PR title follows the convention we established conventional-commit
  • I performed a functional test on my final commit

When you dont install either prisma or next-auth, there is no need to have environment variables, so we don't need to install the boilerplate for the type-safe env-vars in this case.


@nexxeln nexxeln merged commit f5f4f62 into t3-oss:main Jul 15, 2022
devvianto605 pushed a commit to devvianto605/create-devviantex-nextjs-app-deprecated that referenced this pull request Jun 9, 2024
* fix: conditionally run envVars installer

* fix: run prettier

Co-authored-by: Shoubhit Dash <[email protected]>
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