Skip to content

Use the standard prefix VITE_ for Vite env vars#390

Merged
brianhelba merged 1 commit into
masterfrom
vite-env
May 22, 2026
Merged

Use the standard prefix VITE_ for Vite env vars#390
brianhelba merged 1 commit into
masterfrom
vite-env

Conversation

@brianhelba
Copy link
Copy Markdown
Collaborator

VUE_APP_ was used in Vue CLI, but Vite uses only the prefix VITE_. The extra APP_ is pointless.

`VUE_APP_` was used in Vue CLI, but Vite uses only the
prefix `VITE_`. The extra `APP_` is pointless.
@brianhelba brianhelba requested a review from annehaley May 22, 2026 19:28
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 22, 2026

Deploying geodatalytics with  Cloudflare Pages  Cloudflare Pages

Latest commit: 37e7236
Status: ✅  Deploy successful!
Preview URL: https://22ebd329.geodatalytics.pages.dev
Branch Preview URL: https://vite-env.geodatalytics.pages.dev

View logs

@brianhelba
Copy link
Copy Markdown
Collaborator Author

Note, the Terraform changes need to be applied first (the failed build is an intended consequence of the new build check in this PR).

I can update CF Pages manually to get the build successfully passing, and to ensure there are no race conditions in deployment.

@brianhelba brianhelba merged commit 42ed577 into master May 22, 2026
4 checks passed
@brianhelba brianhelba deleted the vite-env branch May 22, 2026 20:40
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