Skip to content

fix: simplify symbol merging logic#3169

Merged
mrlubos merged 1 commit intomainfrom
fix/core-symbol-merge
Jan 2, 2026
Merged

fix: simplify symbol merging logic#3169
mrlubos merged 1 commit intomainfrom
fix/core-symbol-merge

Conversation

@mrlubos
Copy link
Member

@mrlubos mrlubos commented Jan 2, 2026

No description provided.

@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 Jan 2, 2026

🦋 Changeset detected

Latest commit: 77bf81a

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/codegen-core 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 Jan 2, 2026

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

Project Deployment Review Updated (UTC)
hey-api-docs Ready Ready Preview, Comment Jan 2, 2026 5:06pm

@mrlubos mrlubos marked this pull request as ready for review January 2, 2026 16:50
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug 🔥 Broken or incorrect behavior. labels Jan 2, 2026
@mrlubos mrlubos force-pushed the fix/core-symbol-merge branch from 8fd1d5e to bf89868 Compare January 2, 2026 16:57
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 2, 2026

Open in StackBlitz

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

commit: 77bf81a

@mrlubos mrlubos force-pushed the fix/core-symbol-merge branch from bf89868 to 77bf81a Compare January 2, 2026 17:06
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jan 2, 2026
@codecov
Copy link

codecov bot commented Jan 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.09%. Comparing base (24d8b03) to head (77bf81a).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3169   +/-   ##
=======================================
  Coverage   33.09%   33.09%           
=======================================
  Files         440      440           
  Lines       34420    34420           
  Branches     2212     2212           
=======================================
  Hits        11393    11393           
  Misses      22997    22997           
  Partials       30       30           
Flag Coverage Δ
unittests 33.09% <ø> (ø)

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 4eb6d1f into main Jan 2, 2026
17 of 24 checks passed
@mrlubos mrlubos deleted the fix/core-symbol-merge branch January 2, 2026 17:20
@hey-api hey-api bot mentioned this pull request Jan 2, 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:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant