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

Records support, Java 17 and jigsaw modules #325

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

Conversation

Brixomatic
Copy link

  • based on noahra/java to have java record class support (splendid job!)
  • Updated to Java 17 (LTS)
  • Added module-info.java
  • updated maven dependencies
  • added versions-maven-plugin and maven-enforcer-plugin to be notified of newer dependency versions without dependabot.
  • did not fix test cases that were broken from the start

mxyns and others added 22 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 (noahra#7)
actually already works
test - fix added assertion (#3)
test - fix added assertion (#3)
closes #3
…a#13)

* tmp - record investigation / debugging

* test - empty record read

* test - tests showing different object decoder implementation use

Co-authored-by: Maxou <5208681+mxyns@users.noreply.github.com>
Co-authored-by: xinmiao <>
test - add tests for any wrap and stream (noahra#15)
test - add tests for any wrap and stream (noahra#15)
closes noahra#15
@Brixomatic
Copy link
Author

@svobol13 @taowen is there any chance this PR will be merged?

@ozkanpakdil
Copy link

+1

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