search

Web-Dev-Path / web-dev-path

The Web Dev Path platform. Progressive Web App (PWA). Next.js rules!
https://webdevpath.co
GNU General Public License v3.0
33 stars 11 forks source link

Update gitignore to include .env.development #193

Closed cherylli closed 1 year ago

cherylli commented 1 year ago
Web Dev Path

Have you updated the CHANGELOG.md file? If not, please do it.

Yes

What is this change?

Add .env*.local to .gitignore to allow developers to have an extra sets of environment variables for testing

https://nextjs.org/docs/pages/building-your-application/configuring/environment-variables

next.js will load and prioritize .env.development(.local) over .env /.env.local when yarn dev is run

Were there any complications while making this change?

No

How did you verify this change?

Run locally

When should this be merged?

After review

netlify[bot] commented 1 year ago

Deploy Preview for webdevpathstage ready!

Name Link
Latest commit 4a321d553348b525573a6b43e05dfb8b107f0020
Latest deploy log https://app.netlify.com/sites/webdevpathstage/deploys/64e9f3ed41b3910008bc0ac2
Deploy Preview https://deploy-preview-193--webdevpathstage.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

cherylli commented 1 year ago

As per next.js docs, development envrionment variables can be put in .env.developement.local Not sure if we also want to change our project setup to match next.js recommendation, but for now just adding .env.developement.local will do

image https://nextjs.org/docs/pages/building-your-application/configuring/environment-variables#default-environment-variables