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

Basic support of Java 16 records #312

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

mxyns
Copy link

@mxyns mxyns commented Mar 6, 2022

This pull request tries to address the issue #302

This solution uses the Record's canonical constructor by default to create instances of the Records. Other constructors can be used
with @JsonCreator and @JsonProperty (see added tests in src/test/java/com/jsoniter/TestRecord.java).

mxyns and others added 16 commits March 2, 2022 09:54
feat - migrated maven project to java 16 (#1)
test - failing test for unsupported Record (#3)
test - add serialization test (#7)
actually already works
test - add serialization test (#7)
test - fix added assertion (#3)
test - fix added assertion (#3)
closes #3
* tmp - record investigation / debugging

* test - empty record read

* test - tests showing different object decoder implementation use

Co-authored-by: Maxou <[email protected]>
Co-authored-by: xinmiao <>
test - add tests for any wrap and stream (#15)
test - add tests for any wrap and stream (#15)
closes #15
@Brixomatic
Copy link

I've included this in my pull request #325 thanks a lot!

@schrepfler
Copy link

Any chance this can be accepted @svobol13 and released? I've noticed the fempoframework is maintaining a fork with a bit of additional work, perhaps @shaoxt can pick up some maintainer responsibilities?

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.

5 participants