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

fix: fallback on 2xx instead of hardcoded 200 for the ResponseSchema #458

Conversation

stijnvanhulle
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Oct 8, 2023

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

Name Status Preview Comments Updated (UTC)
kubb ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 8, 2023 0:25am

@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 8, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 84686f0:

Sandbox Source
advanced-pet-store Configuration
faker-pet-store Configuration
msw-pet-store Configuration
react-query-pet-store Configuration
typescript-pet-store Configuration
simple-single-pet-store Configuration
solid-query-pet-store Configuration
svelte-query-pet-store Configuration
swr-pet-store Configuration
vue-query-pet-store Configuration
zod-pet-store Configuration
zodios-pet-store Configuration
client-pet-store Configuration

@codecov
Copy link

codecov bot commented Oct 8, 2023

Codecov Report

Merging #458 (84686f0) into main (29d57bd) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #458   +/-   ##
=======================================
  Coverage   85.56%   85.57%           
=======================================
  Files          86       86           
  Lines        5592     5593    +1     
  Branches      696      696           
=======================================
+ Hits         4785     4786    +1     
  Misses        806      806           
  Partials        1        1           
Files Coverage Δ
...kages/swagger/src/generators/OperationGenerator.ts 90.09% <100.00%> (+0.03%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@stijnvanhulle stijnvanhulle merged commit d628687 into main Oct 8, 2023
@stijnvanhulle stijnvanhulle deleted the 456-incorrect-response-type-promiseany-on-the-generated-axios-requests branch October 8, 2023 12:32
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.

Incorrect response type Promise<any> on the generated axios requests
1 participant