Skip to content

Enviroment name validation#11951

Merged
jamesopstad merged 4 commits intomainfrom
james/environment-name-validation
Jan 16, 2026
Merged

Enviroment name validation#11951
jamesopstad merged 4 commits intomainfrom
james/environment-name-validation

Conversation

@jamesopstad
Copy link
Contributor

Add validation for environment name collisions and improve error message for missing environments.

This addresses #11879 (comment)


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: validation change

A picture of a cute animal (not mandatory, but encouraged)

@jamesopstad jamesopstad requested a review from a team as a code owner January 16, 2026 12:32
@changeset-bot
Copy link

changeset-bot bot commented Jan 16, 2026

🦋 Changeset detected

Latest commit: 6955d38

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Jan 16, 2026
@jamesopstad
Copy link
Contributor Author

Thanks for the review @emily-shen.

@jamesopstad jamesopstad merged commit 77fdc18 into main Jan 16, 2026
43 of 45 checks passed
@jamesopstad jamesopstad deleted the james/environment-name-validation branch January 16, 2026 15:06
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Jan 16, 2026
petebacondarwin pushed a commit to ksawaneh/workers-sdk that referenced this pull request Feb 3, 2026
* Validate environment names for all Worker environments

* Add tests

* Add more helpful error message when module runner is not found

* Add changeset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants