-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Kotlin-Spring] support to Spring boot3 & jakarta extension #13620
[Kotlin-Spring] support to Spring boot3 & jakarta extension #13620
Conversation
@@ -385,4 +385,33 @@ public void beanQualifiers() throws Exception { | |||
"@RestController(\"org.openapitools.api.PingApiController\")" | |||
); | |||
} | |||
|
|||
@Test(description = "use Spring boot 3 & jakarta extension") | |||
public void useSpringBoot3() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see several project examples inside samples/server/petstore but I don't see any example with jakarta, is that intended?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these file changes are intended as i modified import style for existing ones. unit test is covering jakarta imports validation and not sure on what basis framework generate sample project for different types
cc @jimschubert (2017/09) ❤️, @dr4ke616 (2018/08) @karismann (2019/03) @Zomzog (2019/04) @andrewemery (2019/10) @4brunu (2019/11) @yutaka0m (2020/03) |
Looking forward to 6.3.0 🙂 |
<parent> | ||
<groupId>org.springframework.boot</groupId> | ||
<artifactId>spring-boot-starter-parent</artifactId> | ||
<version>3.0.0-M5</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be 3.0.0
instead of 3.0.0-M5
. Then I think also the Spring milestone repository can be removed.
looking forward to 6.3.0 |
Any plans for this to be merged ? I have successfully tested this PR using multiple different projects. Anything missing here were I could help ? |
We are also looking forward to the upcoming release for migrating our Kotlin service to Spring Boot 3. |
@jayandran-Sampath do you need some help with merging? |
Also looking forward to this feature :) |
Hey thanks for checking.. it would be great if you or anyone else help me to resolve conflicts.. due to personal reason, i could not work on this for now.. thanks |
Your commits are meged. I think you can close this PR. |
Updated moustache files to support jakarta extension & spring boot 3
refer #13578
PR checklist
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*
.For Windows users, please run the script in Git BASH.
master
(6.1.0) (minor release - breaking changes with fallbacks),7.0.x
(breaking changes without fallbacks)