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

feat: restrict characters useable for environment name #18255

Conversation

sapphi-red
Copy link
Member

Description

Related discussion: #16471 (comment)

@sapphi-red sapphi-red added the feat: environment API Vite Environment API label Oct 2, 2024
Copy link

stackblitz bot commented Oct 2, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@patak-dev patak-dev merged commit 9ab6180 into vitejs:main Oct 2, 2024
11 of 12 checks passed
@sapphi-red sapphi-red deleted the feat/restrict-characters-useable-for-environment-name branch October 2, 2024 06:41
@sapphi-red
Copy link
Member Author

@hi-ogawa It seems this PR broke vite-environment-examples because it uses react-server as an environment name.
https://github.com/vitejs/vite-ecosystem-ci/actions/runs/11158387830/job/31014572567#step:8:1408
Do you if it's fine to rename it on your side?

@hi-ogawa
Copy link
Collaborator

hi-ogawa commented Oct 3, 2024

Oh, I didn't know - is also removed. I don't mind updating my example to avoid - at all, but that feels a bit too much restriction to be honest (not sure maybe this is just because I've been writing react-server for a long time).

@hi-ogawa
Copy link
Collaborator

hi-ogawa commented Oct 3, 2024

Well, actually it's not really an issue for me. I just updated mine as it was too long to write hi-ogawa/vite-environment-examples#129

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: environment API Vite Environment API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants