Mathew Polzin

Results 225 comments of Mathew Polzin

Strange, when testing against OpenAPIKit directly, I actually get results that would imply the opposite expected result (I would expect with quotes to work and without quotes to fail): ```...

Sanity checking further, if I use the whole snippets from the OP, I get a schema that has `nullable = true` when using the quotes (expected) and I get `nullable...

I'm not trying to say the bug report is unreproducible, of course, I am just not seeing any obvious problems in OpenAPIKit's handling of the given schemas yet.

Yams behavior might have changed meaningfully RE this situation, but the results I saw above seemed to indicate that not only did Yams parse the schema alright but OpenAPIKit also...

@TopIvanAbramov do you have time to try locking down Yams to version 5.0.0 and see if that fixes the problem? If it does, at least we know where to look....

I re-ran my small test case directly against OpenAPIKit using Yams 5.0.6 rather than 5.1.0 and (like you) observed a difference. With the previous release of Yams, I get the...

@TopIvanAbramov could you elaborate on the correct and incorrect behavior for this ticket? When things work as expected, what is being generated? I'm wondering if the desired behavior occurs when...

Thanks for the PR. I’m on a trip for the next two weeks so unless someone else jumps in there will be a bit of a delay in looking this...

Hey! Sorry, I kept not thinking of this PR when I had time to look at it because it had been removed from my review queue after my previous comments....

> > This looks pretty good. I only have one inline comment. > > I do have one high level question, though. Is it better (or more versatile) or just...