Jachym Metlicka

Results 24 comments of Jachym Metlicka

> Thanks for the PR > > I'm no expert in Kotlin. > > Regarding the following: > > ``` > override val nonRequiredWithDefaultString: kotlin.String? = "defaultValue" > ``` >...

> ~- Some users might use the server side models for testing purposes, where they supply null values to optional parameters. Although you can argue that they should generate client...

I am wondering if I should create a corresponding issue and link the PR from there? If we expect further discussion, maybe the issue would be a better place? Don't...

Sorry for tagging you directly @wing328. I am just wondering if there is anything I can do to improve the chances of getting this merged. And @stefankoppier - I take...

As agreed on Slack, I will modify this PR to control this behavior conditionally via _flag_

Hello @Chrimle, I am not a maintainer, but this change looks sensible to me. I really like that you changed it _everywhere_ even though it might theoretically not cause an...

@Chrimle - I kind of understand the intention with limiting the scope of the changes. But where do you see risk with inlining everywhere? I don't think the issues in...

I would expect the same issues in the generated API classes (controllers/interfaces/delegates) when e.g. using a _custom user-defined_ schema called `Locale` as a request or response body.

As far as kotlin-spring is concerned, I tried to do as thorough job as possible here: #22449

I will try to take a look at this when I have time. I see there are many places in kotlin generators where this is an issue. Basically any time...