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
Change env var prefix #2308
Comments
Do we have to do something? I mean, if it is on the vite config we just need to bump the version of vite right? |
yeah. and I think we should change our default and document it |
Maybe |
I sent a PR to accept both |
Isn't this just an extra thing people have to learn? If someone knows Vite (and to become expert in SvelteKit, they will eventually need to know their way around Vite, at least a bit) then they will know about By contrast I don't think I would immediately know what |
It is an extra thing to learn to some extent, but if you don't know about |
Is there a specific reason why you suggest I agree that, though the reasons make sense, it has always been a little weird to put |
I didn't know that was becoming a pattern. I hadn't seen |
I'm going to go ahead and close this. Everyone's got a bit different opinion about it and it would make the behavior diverge from VITE which might add as much confusion as it would solve. If you'd like to configure this, you can do so in your |
This may have already shipped (?), but I'd opine that something like |
this was left as |
Describe the problem
Vite will now let you change
VITE
env var prefix to be something else likePUBLIC
orCLIENT
Describe the proposed solution
vitejs/vite#4676
Alternatives considered
No response
Importance
nice to have
Additional Information
No response
The text was updated successfully, but these errors were encountered: