Skip to content
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 code generator #2598

Merged
merged 5 commits into from
Oct 2, 2019
Merged

Conversation

tristanlins
Copy link
Contributor

Kotlin Generator based / as a copy of the Java Generator.

When generating Kotlin code, there were significant differences, so I built a new generator and did not make the Java Generator switchable.

BTW: Thanks for this great project 🤗

@dotansimha
Copy link
Owner

This is awesome! Thank you so much @tristanlins ! We'll review it very soon :)

Propagate the default values into kotlin.
The most benefit is in unit tests.
Let is not necessary, for non-null values.
Reduce compiler warnings.
@tristanlins
Copy link
Contributor Author

After using the plugin myself in a productive project, a few bumps have been noticed.

I have revised it again.

@dotansimha
Copy link
Owner

Awesome @tristanlins ! Thank you!
I added 2 minor comments, but other than that - it looks great.

@dotansimha dotansimha merged commit 5634e43 into dotansimha:master Oct 2, 2019
@dotansimha
Copy link
Owner

Merged, thank you @tristanlins !

@tristanlins tristanlins deleted the feature-kotlin branch October 2, 2019 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants