Improve Hyperdrive error messages for missing required options#14228
Conversation
🦋 Changeset detectedLatest commit: e928ea8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
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 |
|
✅ All changesets look good |
|
All the option names referenced in the error messages match the actual yargs option definitions ( I notice one subtle point: the Let me confirm the changeset frontmatter is correctly formatted and double-check there's nothing relating to D1 (the branch name mentions D1 but the PR is about Hyperdrive). The branch is named Let me verify the changeset content claims about The changes are straightforward error-message wording improvements with matching test snapshot updates. Option names are accurate, supported scheme values match the LGTM |
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-auth
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
|
Codeowners approval required for this PR:
Show detailed file reviewers |
petebacondarwin
left a comment
There was a problem hiding this comment.
I agree with Devin but otherwise LGTM
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
|
@petebacondarwin would you mind having a quick look on the last two commits I've pushed here? 🙂🙏 |
This PR simply improves some hyperdrive error messages to make them more clear and actionable.
A picture of a cute animal (not mandatory, but encouraged)