Skip to content

Update @hey-api/openapi-ts and move to pnpm catalog#12828

Merged
jamesopstad merged 2 commits intomainfrom
james/hey-api
Mar 10, 2026
Merged

Update @hey-api/openapi-ts and move to pnpm catalog#12828
jamesopstad merged 2 commits intomainfrom
james/hey-api

Conversation

@jamesopstad
Copy link
Contributor

@jamesopstad jamesopstad commented Mar 10, 2026

Fixes #12726

Update @hey-api/openapi-ts and move to pnpm catalog


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

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


Open with Devin

@jamesopstad jamesopstad requested a review from a team as a code owner March 10, 2026 09:30
@changeset-bot
Copy link

changeset-bot bot commented Mar 10, 2026

🦋 Changeset detected

Latest commit: 357266c

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

@workers-devprod
Copy link
Contributor

workers-devprod commented Mar 10, 2026

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@github-actions
Copy link
Contributor

⚠️ Issues found

File: .changeset/late-humans-win.md

Problem: Insufficient changeset description

The description Update @hey-api/openapi-ts to ^0.94.0 is a "bad example" pattern explicitly called out in .changeset/README.md:

"update dependency" - Which one? Why? Any user impact?

The changeset should explain:

  • Why this dependency is being updated (bug fix? new feature needed? security patch?)
  • Whether there is any user-facing impact from the update
  • If there is no user-facing impact, consider whether a changeset is needed at all (purely internal dependency bumps with no user impact don't require changesets)

Version type: patch is acceptable if this is a routine dependency update with no user-visible changes.

Suggested fix: Either remove the changeset if there is truly no user-facing impact, or expand the description to explain the reason and impact, e.g.:

Update `@hey-api/openapi-ts` to ^0.94.0

This dependency update brings in [reason, e.g. bug fixes / new codegen features / security patches]. [Any user-visible impact or "No user-facing changes."]

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 10, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12828

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12828

miniflare

npm i https://pkg.pr.new/miniflare@12828

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12828

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12828

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12828

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12828

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12828

wrangler

npm i https://pkg.pr.new/wrangler@12828

commit: 7801626

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Mar 10, 2026
@jamesopstad jamesopstad merged commit cb14820 into main Mar 10, 2026
41 of 44 checks passed
@jamesopstad jamesopstad deleted the james/hey-api branch March 10, 2026 10:52
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

3 participants