Contribution proposal: is there somebody intereste...
# spring
s
Contribution proposal: is there somebody interested to translate Spring Boot code sample to Kotlin like I did on the Spring Framework refdoc? The refdoc is long but there are not so many code samples so I do think it is doable in a reasonable amount of time.
i
I am!
s
Feel free to sync with @corneil to share the work if needed
c
I'll spend some time to get a first one in
s
Make sense
i
@sdeleuze what branch do I start working from? master?
s
Yes it is master, but I would suggest waiting for Corneil branch is order to avoid duplicated effort
i
I’ve synced with him, he said I can go ahead and he’ll only be able to spend much time until later today
s
Oh great then
So yes Boot master branch
Tiny tricks, when you have annotated code with
<1>
You have to duplicate code comments for both Java and Kotlin and put an empty line between both (like I did on Framework).
There may be style improvement to do but don't care for now
I will see that with Boot team
i
I don’t understand what you mean by
when you have annotated code with
<1>
I’ll submit a PR soon so I don’t go too far off
I’m starting with the using-spring-boot.adoc file
👍 1
i
Okay, I get now. Thanks
Do I need to reference spring-projectsgh-21778 in my commits?
s
Nope, just in the PR comment
i
Ok then
s
@ienoobong @corneil I have seen the runnable code new commit 👍 How do you plan to move forward, have you already defined who do what to translate the code samples ? Any estimation of how many time will be needed ?
c
If this is the right track we will start moving forward and provide feedback in a day or so
👍 1
i
Hi @sdeleuze, I’ve completed for
using-spring-boot.adoc
https://github.com/spring-projects/spring-boot/pull/18017 Would be great to get a review
All green!
s
I have done a review, if you can't find how to avoid setting the Kotlin version please ask to Andy in the PR comments
Thanks for taking in account my review.
@ienoobong is it possible to give credentials to @corneil on your Boot fork in order to allow him to contribute some part of the Java -> Kotlin ? Could you split the work between both of you now that we have a stable basis ?
i
Hi @sdeleuze yes, I’ve added him to the fork. We’ll get back on the work splitting soonest.
s
Great, thanks a lot for this great contribution
🤗 1
I am a little bit pushee in order to let the time to Boot team to review and integrate it in order to be able to announce it at SpringOne Platform 😉
Of course I will mention your names
i
No worries. I can understand. When is SpringOne coming up?
Less than a month away!
s
October 7th but we will travel before and Boot team need some time to review and merge it. Ideally the PR should be merged in Boot 2.2 RC1 expected September 26th. https://github.com/spring-projects/spring-boot/milestone/144
If that's possible of course
Please let me know if you think that's doable after discussing with @corneil
i
Ok we will