fastify / fastify-swagger

Swagger documentation generator for Fastify
MIT License
910 stars 200 forks source link

Don't use JSONObject for `transform`. #726

Closed A5rocks closed 1 year ago

A5rocks commented 1 year ago

Checklist

I'll use CI to make sure npm run test and npm run benchmark are AOK. I can elaborate on why this change is necessary, but instead just look at the changes to the tests. BTW the example in the readme still works (once you change it by making transformedSchema be ...ts then const transformedSchema = ts as FastifySchema).

A5rocks commented 1 year ago

Hi! Maybe the Windows flakiness is gone now?

mcollina commented 1 year ago

This can land! thx

A5rocks commented 1 year ago

Thank you!