OpenAPITools / openapi-generator

OpenAPI Generator allows generation of API client libraries (SDK generation), server stubs, documentation and configuration automatically given an OpenAPI Spec (v2, v3)
https://openapi-generator.tech
Apache License 2.0
20.63k stars 6.29k forks source link

[kotlin-spring] Fix two missing ) in kotlin-spring/pathParams.mustache (#18854) #18861

Closed Humle-T closed 1 month ago

Humle-T commented 1 month ago

Fixes #18854

PR checklist

I've added two missing ) to the template found in modules/openapi-generator/src/main/resources/kotlin-spring/pathParams.mustache.

In the issue #18854 there is an example openapi.yaml linked in a Gist, which can be used to generate a now compileable Kotlin class.

I ran the above commands for generating samples and docs, but this didn't change any files (I guess this particular use case is not covered in the current examples).

A small note: In the issue linked, I provided suggestion for a fix, which closely resembles the changes in this PR - but not quite. I believe that my first suggestion was not quite right, and this PR has the )'s in the correct places.

Mentions at the Kotlin "Members of Technical Committee": @jimschubert @dr4ke616 @karismann @Zomzog @andrewemery @4brunu @yutaka0m @stefankoppier

Humle-T commented 1 month ago

Thank you for quick and efficient issue/PR handling! 👍

4brunu commented 1 month ago

Thanks for creating the PR with the fix 👍