Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add OpenAPI 2.0.x experimental parser #1525

Merged
merged 2 commits into from
Jan 6, 2025
Merged

Conversation

mrlubos
Copy link
Member

@mrlubos mrlubos commented Jan 1, 2025

Closes #1524

Copy link

stackblitz bot commented Jan 1, 2025

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

Copy link

changeset-bot bot commented Jan 1, 2025

🦋 Changeset detected

Latest commit: 3b01857

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

This PR includes changesets to release 11 packages
Name Type
@hey-api/openapi-ts Minor
@hey-api/client-fetch Minor
@hey-api/client-axios Minor
@hey-api/docs Minor
@example/openapi-ts-axios Patch
@example/openapi-ts-fastify Patch
@example/openapi-ts-fetch Patch
@example/openapi-ts-tanstack-angular-query-experimental Patch
@example/openapi-ts-tanstack-react-query Patch
@example/openapi-ts-tanstack-svelte-query Patch
@example/openapi-ts-tanstack-vue-query 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

Copy link

vercel bot commented Jan 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hey-api-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 6, 2025 8:09am

Copy link

codecov bot commented Jan 1, 2025

Codecov Report

Attention: Patch coverage is 18.16133% with 1816 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@ff29d7b). Learn more about missing BASE report.
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...ages/openapi-ts/src/openApi/2.0.x/parser/schema.ts 6.42% 670 Missing ⚠️
...s/openapi-ts/src/openApi/2.0.x/parser/operation.ts 8.28% 332 Missing ⚠️
...kages/openapi-ts/src/openApi/2.0.x/parser/index.ts 7.43% 224 Missing ⚠️
.../openapi-ts/src/plugins/@hey-api/schemas/plugin.ts 3.54% 136 Missing ⚠️
...s/openapi-ts/src/openApi/2.0.x/parser/parameter.ts 15.82% 133 Missing ⚠️
.../openapi-ts/src/openApi/2.0.x/parser/pagination.ts 7.27% 102 Missing ⚠️
...ages/openapi-ts/src/plugins/@hey-api/sdk/plugin.ts 25.77% 72 Missing ⚠️
...s/openapi-ts/src/openApi/2.0.x/parser/mediaType.ts 28.57% 50 Missing ⚠️
packages/openapi-ts/src/compiler/types.ts 5.88% 32 Missing ⚠️
...ages/openapi-ts/src/openApi/3.0.x/parser/schema.ts 26.31% 14 Missing ⚠️
... and 8 more
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1525   +/-   ##
=======================================
  Coverage        ?   59.54%           
=======================================
  Files           ?      158           
  Lines           ?    24554           
  Branches        ?     1880           
=======================================
  Hits            ?    14620           
  Misses          ?     9927           
  Partials        ?        7           
Flag Coverage Δ
unittests 59.54% <18.16%> (?)

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.

Copy link

pkg-pr-new bot commented Jan 1, 2025

Open in Stackblitz

npm i https://pkg.pr.new/@hey-api/client-axios@1525
npm i https://pkg.pr.new/@hey-api/client-fetch@1525
npm i https://pkg.pr.new/@hey-api/openapi-ts@1525

commit: 3b01857

@mrlubos mrlubos force-pushed the feat/openapi-2.0.0-parser branch from f45dda6 to 3b01857 Compare January 6, 2025 08:08
@mrlubos mrlubos marked this pull request as ready for review January 6, 2025 08:08
@mrlubos mrlubos merged commit 076bf07 into main Jan 6, 2025
17 checks passed
@mrlubos mrlubos deleted the feat/openapi-2.0.0-parser branch January 6, 2025 08:16
@github-actions github-actions bot mentioned this pull request Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OpenAPI 2.0 parser tasks
1 participant