Alexandre Stahmer

Results 85 comments of Alexandre Stahmer

also a "contributing" with a link to changeset could be nice

hey ! glad it heps ! 🙏 can you provide a minimal repro of that bug ? it seems very weird to me that it's fixed with `complexity-threshold` 😅 the...

ah yes, makes sense now this is the option you're looking for then (and this is the default value): ![Screenshot 2023-01-29 at 19 00 48](https://user-images.githubusercontent.com/47224540/215346200-225c537e-d404-4d0e-8970-c9b6ae69ef22.png) `--media-type-expr Pass an expression to...

this looks like it still exists in zodios side https://github.com/ecyrbe/zodios/blob/bc05f0fbfe51dd7db3fd89b27e3a92ed0c1c8559/website/docs/client/client.md?plain=1#L22 you can use a custom template to remove this import / the `createApiClient`

it's probably outdated, I guess it was supposed to be `gen:samples` to contribute, you should a specific test for the case you add/update and just check that the other tests...

yeah it's probably outdated at this point, feel free to open a PR to use SwaggerParser types 🙏

hey, yes you are right ! looking at the implementation, it should be nullish https://github.com/astahmer/openapi-zod-client/blob/b4e5af17c67e3a4d4a04f96dec6145cd4ee3e7cb/src/openApiToZod.ts#L198-L209 and those tests : https://github.com/astahmer/openapi-zod-client/blob/c95b2350ec2f02c441ff755abedc6302f667d702/lib/src/openApiToZod.test.ts#L110-L120 are not up to date, they should look like this...

my bad I just noticed the notification for this PR, could you resolve the conflicts so I can merge ? also, can you add a changeset ?

good idea ! feel free to open a PR for that (with a [changeset](https://github.com/changesets/changesets/blob/main/docs/intro-to-using-changesets.md))