Skip to content

Conversation

warrenseine
Copy link
Contributor

Watch mode was broken in 0.85.0 when #2602 was introduced.

Here's a fix proposal. Happy to iterate, as I definitely don't have the full context regarding the "multiple configs" feature.

Copy link

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

Copy link

changeset-bot bot commented Oct 12, 2025

🦋 Changeset detected

Latest commit: b8f7ac4

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

This PR includes changesets to release 1 package
Name Type
@hey-api/openapi-ts Patch

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

Copy link

vercel bot commented Oct 12, 2025

@warrenseine is attempting to deploy a commit to the Hey API Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Oct 12, 2025
@dosubot dosubot bot added the bug 🔥 Something isn't working label Oct 12, 2025
Copy link

codecov bot commented Oct 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.14%. Comparing base (227251d) to head (b8f7ac4).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2751   +/-   ##
=======================================
  Coverage   29.14%   29.14%           
=======================================
  Files         391      391           
  Lines       38093    38093           
  Branches     2272     2272           
=======================================
  Hits        11104    11104           
  Misses      26959    26959           
  Partials       30       30           
Flag Coverage Δ
unittests 29.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

pkg-pr-new bot commented Oct 12, 2025

Open in StackBlitz

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/codegen-core@2751
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/nuxt@2751
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/openapi-ts@2751
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/vite-plugin@2751

commit: 3293cad

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Oct 13, 2025
Copy link
Member

@mrlubos mrlubos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 13, 2025
@mrlubos mrlubos merged commit 617f3a3 into hey-api:main Oct 13, 2025
14 of 15 checks passed
@hey-api hey-api bot mentioned this pull request Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Something isn't working lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants