-
Notifications
You must be signed in to change notification settings - Fork 230
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
Remove now.json
in favor of built-in env vars support
#1362
Comments
I just have to go through and manually add all of these to my Vercel dashboard though, yeah? Or does having the secrets set already make this work out of the box? Right now I see this in the dashboard: But of course I've already done |
Convert - it's converting them from just secrets to env variables. When you select "secret", you'll be able to find all the existing secrets in the dropdown. The biggest benefit is being able to run If you hit any issues, please let me know! |
@leerob I just closed this after making the changes, but now I'm wondering: is there a best practice for using GitHub actions to run CI with whatever the |
👋 I would suggest running Cypress again the preview deployment URL. That's what we do at Vercel 😄 |
Oh wow, that's quite smart actually – any chance you are able to share or publish some of those action workflow files? Otherwise I can probably figure out how to string that together so things run in the correct order. |
Hey 😄 You might already know this, but I figured I'd mention it. You can remove
now.json
entirely with environment variables support first-class through dotenv in Next.js. You can also removeenv
fromnext.config.js
!Also - Inside Vercel, you can now more easily choose if your env var is a secret or plaintext.
The text was updated successfully, but these errors were encountered: