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

Add three extensions for JsonToKotlinClass plugin #349

Merged
merged 4 commits into from
May 22, 2021

Conversation

varenyzc
Copy link

CamelCaseSupport: It can type camel-case property.
BuildFromJsonSupport : It can type a companion object with a function what can build from json.
NeedNonNullableClassesSupport: It can make the classes that you type in the extensions input area non nullabe.

@wuseal
Copy link
Owner

wuseal commented May 18, 2021

@varenyzc Greate! Amazing me, you do these so fast 😱

Also we need unit test for each new feature.

Please make up the test cases. Thank you 😁

Can ref: src/test/kotlin/extensions/nstd/ReplaceConstructorParametersByMemberVariablesSupportTest.kt

@varenyzc
Copy link
Author

@wuseal okay,i will make it tomorrow😁

@varenyzc
Copy link
Author

@wuseal I have completed the test case. Should I make a new pull request?

@wuseal
Copy link
Owner

wuseal commented May 20, 2021

@varenyzc No need, You could just push your test cases to branch varenyzc:master is OK

Signed-off-by: yuanzicong <[email protected]>
@wuseal
Copy link
Owner

wuseal commented May 22, 2021

@varenyzc Well done! Thanks very much that you provided these awesome features for JTK👏👏👏👏👍

@wuseal wuseal self-requested a review May 22, 2021 09:07
Copy link
Owner

@wuseal wuseal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wuseal wuseal merged commit 03567c3 into wuseal:master May 22, 2021
@wuseal wuseal added this to the 3.7.1 milestone May 22, 2021
@wuseal wuseal added awesome pull request very good pull request for JsonToKotlinClass enhancement L4 Difficulty level 4 (Maximum is 9) labels May 22, 2021
@varenyzc
Copy link
Author

@wuseal This is my first time to make a PR for open source project.Thanks for your review!

@wuseal
Copy link
Owner

wuseal commented May 25, 2021

Published in 3.7.1-EAP-3

wuseal added a commit that referenced this pull request Sep 20, 2021
Add three extensions for JsonToKotlinClass plugin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awesome pull request very good pull request for JsonToKotlinClass L4 Difficulty level 4 (Maximum is 9)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants