这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@mrlubos
Copy link
Member

@mrlubos mrlubos commented Sep 10, 2025

Fixes #2603

@bolt-new-by-stackblitz
Copy link

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

@changeset-bot
Copy link

changeset-bot bot commented Sep 10, 2025

🦋 Changeset detected

Latest commit: 1193c4d

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

@vercel
Copy link

vercel bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hey-api-docs Ready Ready Preview Comment Sep 10, 2025 0:14am

@mrlubos mrlubos marked this pull request as ready for review September 10, 2025 12:13
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Sep 10, 2025
@dosubot dosubot bot added the bug 🔥 Something isn't working label Sep 10, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Sep 10, 2025

Open in StackBlitz

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

commit: 1193c4d

@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 23.83%. Comparing base (a82ba4d) to head (1193c4d).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
packages/openapi-ts/src/config/parser.ts 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2623      +/-   ##
==========================================
- Coverage   23.83%   23.83%   -0.01%     
==========================================
  Files         379      379              
  Lines       36930    36933       +3     
  Branches     1626     1626              
==========================================
  Hits         8803     8803              
- Misses      28114    28117       +3     
  Partials       13       13              
Flag Coverage Δ
unittests 23.83% <0.00%> (-0.01%) ⬇️

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.

@mrlubos mrlubos merged commit 0cb4db6 into main Sep 10, 2025
16 of 17 checks passed
@mrlubos mrlubos deleted the fix/config-parser-hooks branch September 10, 2025 12:23
@hey-api hey-api bot mentioned this pull request Sep 10, 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 size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hooks not working as documented for old TanStack behavior

2 participants