Skip to content

Conversation

@mrlubos
Copy link
Member

@mrlubos mrlubos commented Dec 24, 2025

Closes #3132

@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 Dec 24, 2025

🦋 Changeset detected

Latest commit: ab80a4b

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

This PR includes changesets to release 2 packages
Name Type
@hey-api/codegen-core Patch
@hey-api/openapi-ts Minor

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

@mrlubos mrlubos marked this pull request as ready for review December 24, 2025 00:15
@vercel
Copy link

vercel bot commented Dec 24, 2025

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

Project Deployment Review Updated (UTC)
hey-api-docs Ready Ready Preview, Comment Dec 26, 2025 11:39pm

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. feature 🚀 Feature request. labels Dec 24, 2025
@codecov
Copy link

codecov bot commented Dec 24, 2025

Codecov Report

❌ Patch coverage is 3.76766% with 1226 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.83%. Comparing base (6d2a873) to head (ab80a4b).
⚠️ Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
...es/openapi-ts/src/plugins/zod/mini/toAst/number.ts 0.86% 114 Missing ⚠️
.../openapi-ts/src/plugins/valibot/v1/toAst/number.ts 1.85% 106 Missing ⚠️
...ages/openapi-ts/src/plugins/zod/v4/toAst/number.ts 0.95% 104 Missing ⚠️
...ages/openapi-ts/src/plugins/zod/v3/toAst/number.ts 0.96% 103 Missing ⚠️
.../openapi-ts/src/plugins/valibot/v1/toAst/string.ts 1.98% 99 Missing ⚠️
...es/openapi-ts/src/plugins/zod/mini/toAst/string.ts 1.04% 95 Missing ⚠️
...ages/openapi-ts/src/plugins/zod/v3/toAst/string.ts 1.07% 92 Missing ⚠️
...ages/openapi-ts/src/plugins/zod/v4/toAst/string.ts 1.12% 88 Missing ⚠️
.../openapi-ts/src/plugins/valibot/v1/toAst/object.ts 2.89% 67 Missing ⚠️
...ages/openapi-ts/src/plugins/zod/v3/toAst/object.ts 1.53% 64 Missing ⚠️
... and 20 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3147      +/-   ##
==========================================
- Coverage   33.28%   32.83%   -0.46%     
==========================================
  Files         429      430       +1     
  Lines       33643    34126     +483     
  Branches     2156     2157       +1     
==========================================
+ Hits        11199    11204       +5     
- Misses      22416    22894     +478     
  Partials       28       28              
Flag Coverage Δ
unittests 32.83% <3.76%> (-0.46%) ⬇️

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.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 24, 2025

Open in StackBlitz

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

commit: ab80a4b

@mrlubos mrlubos self-assigned this Dec 24, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 25, 2025
@mrlubos mrlubos force-pushed the feat/plugin-resolvers-number-2 branch from 7aefadc to 636edb8 Compare December 25, 2025 22:21
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Dec 26, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 26, 2025
@mrlubos mrlubos merged commit a6de8a7 into main Dec 26, 2025
13 of 15 checks passed
@mrlubos mrlubos deleted the feat/plugin-resolvers-number-2 branch December 26, 2025 23:46
@hey-api hey-api bot mentioned this pull request Dec 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 🚀 Feature request. size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resolvers for number types

2 participants