@VishnuSanal is attempting to deploy a commit to the sparckles Team on Vercel.

A member of the Team first needs to authorize it.

CodSpeed Performance Report

Merging #932 will improve performances by ×2.7

_{Comparing VishnuSanal:openapi-response-schema (51b6f29) with main (e59c2ec)}

Summary

⚡ 3 improvements
✅ 111 untouched benchmarks

Benchmarks breakdown

The schema will be defaulted to "any" for now; until typing is implemented. @sansyrox PTAL. I am not sure whether this was the expectation -- CMIIW.

@VishnuSanal , there is also an issue in rendering of the docstrings in the application.

The above is an expected response.

But what happens is the one below

@VishnuSanal , could you fix it?

@VishnuSanal , there is also an issue in rendering of the docstrings in the application.

what is the issue? the location where the description is shown? fastapi might be using a slightly different version of swagger; that must be the reason. should investigate. is this a major breaking feature? isn't it just about the aesthetics?

what is the issue? the location where the description is shown? fastapi might be using a slightly different version of swagger; that must be the reason. should investigate. is this a major breaking feature? isn't it just about the aesthetics?

The issue is that Robyn's docstrings are not legible. We are appending them to the wrong location! This is may be due to a different version of swagger but feels more like wrong openapi.json generation.

Hence, a blocking issue, which needs to be fixed before merging

The docstrings are being appended only to the summary. They should also be added to the description field.

done, PTAL. although unrelated, I have also removed openapi integration from future roadmap in the docs -- hope this is okay & need not create a new PR.

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