Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

support kotlin #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

support kotlin #36

wants to merge 1 commit into from

Conversation

uuidcode
Copy link

querydsl/querydsl#1828

I roughly wrote it.
I added test cases and verified kotlin compiling in many cases.
If this PR is acceptable, I think querydsl-maven-plugin and querydsl-codegen need to be changed.

I love querydsl and I hope querydsl will getting more popular.

thanks.

@Shredder121
Copy link
Member

Ah thanks for spearheading this.
I have some experience writing Kotlin, but I'll have to read the spec before I can help out writing this.
The earliest that will be is this weekend, just so you know what to expect. 😉

@uuidcode uuidcode force-pushed the kotlin branch 5 times, most recently from 188fd84 to 7745d46 Compare April 15, 2016 09:06
@Shredder121
Copy link
Member

Hi, I just wanted to say that in my opinion we could integrate Kotlin support in codegen more easily than Querydsl itself.
Since we'll have to integrate with tooling, we need proper documentation for language features, platform integration, etc.
I'll update the original issue with some comments regarding the situation.
Anyway, thanks again for starting on this, we appreciate it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants