Skip to content

fix: add overloads for defineConfig to improve type inference#3263

Closed
hyoban wants to merge 2 commits intohey-api:mainfrom
hyoban:1-25-type
Closed

fix: add overloads for defineConfig to improve type inference#3263
hyoban wants to merge 2 commits intohey-api:mainfrom
hyoban:1-25-type

Conversation

@hyoban
Copy link
Copy Markdown
Contributor

@hyoban hyoban commented Jan 25, 2026

No description provided.

@bolt-new-by-stackblitz
Copy link
Copy Markdown

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

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 25, 2026

🦋 Changeset detected

Latest commit: 05e5dc4

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
Copy Markdown

vercel Bot commented Jan 25, 2026

@hyoban 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 size:XS This PR changes 0-9 lines, ignoring generated files. bug 🔥 Broken or incorrect behavior. labels Jan 25, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 35.74%. Comparing base (81e98e4) to head (05e5dc4).
⚠️ Report is 110 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3263   +/-   ##
=======================================
  Coverage   35.74%   35.74%           
=======================================
  Files         454      454           
  Lines       34650    34650           
  Branches     2475     2475           
=======================================
  Hits        12385    12385           
  Misses      22236    22236           
  Partials       29       29           
Flag Coverage Δ
unittests 35.74% <ø> (ø)

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.

@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 Jan 25, 2026
@hyoban hyoban marked this pull request as draft January 25, 2026 08:51
@hyoban hyoban marked this pull request as ready for review January 25, 2026 11:26
@mrlubos
Copy link
Copy Markdown
Member

mrlubos commented Feb 5, 2026

I see now why this was needed. I fixed it in another pull request!

@mrlubos mrlubos closed this Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Broken or incorrect behavior. 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