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

Conversation

@mrlubos
Copy link
Member

@mrlubos mrlubos commented Oct 26, 2025

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 Oct 26, 2025

🦋 Changeset detected

Latest commit: 0d32e7b

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

@mrlubos mrlubos marked this pull request as ready for review October 26, 2025 00:42
@vercel
Copy link

vercel bot commented Oct 26, 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 Oct 26, 2025 7:35pm

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 26, 2025
import { getSpecsPath } from '../../utils';
// @ts-ignore
import { myClientPlugin } from './custom/client/plugin';
import { myClientPlugin } from '../packages/openapi-ts-tests/main/test/custom/client/plugin';

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused import myClientPlugin.
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Oct 26, 2025
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Oct 26, 2025
@codecov
Copy link

codecov bot commented Oct 26, 2025

Codecov Report

❌ Patch coverage is 80.08299% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.59%. Comparing base (3292581) to head (0d32e7b).
⚠️ Report is 17 commits behind head on main.

Files with missing lines Patch % Lines
packages/openapi-ts/src/generate.ts 62.00% 38 Missing ⚠️
packages/openapi-ts/src/cli.ts 96.85% 4 Missing ⚠️
packages/openapi-ts/src/generate/legacy/output.ts 0.00% 1 Missing and 1 partial ⚠️
packages/openapi-ts/src/generate/output.ts 60.00% 1 Missing and 1 partial ⚠️
packages/openapi-ts/src/run.ts 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2852      +/-   ##
==========================================
+ Coverage   30.36%   30.59%   +0.22%     
==========================================
  Files         485      488       +3     
  Lines       39803    39938     +135     
  Branches     2413     2442      +29     
==========================================
+ Hits        12088    12219     +131     
- Misses      27684    27685       +1     
- Partials       31       34       +3     
Flag Coverage Δ
unittests 30.59% <80.08%> (+0.22%) ⬆️

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 Oct 26, 2025

Open in StackBlitz

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

commit: 0d32e7b

@mrlubos mrlubos merged commit 3def2b3 into main Oct 26, 2025
17 checks passed
@mrlubos mrlubos deleted the fix/cli-typescript branch October 26, 2025 19:44
@hey-api hey-api bot mentioned this pull request Oct 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants