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

Fixed Record package #17

Merged
merged 1 commit into from
Mar 23, 2023
Merged

Fixed Record package #17

merged 1 commit into from
Mar 23, 2023

Conversation

jeber-ibm
Copy link
Member

Hit a compile error when opening in eclipse. Had to explicitly state the package for Report

@ThePrez
Copy link
Member

ThePrez commented Mar 23, 2023

Interesting. Changes LGTM!

Curiosity is getting me though.... What version of JDK are you using? And what does it conflict with?

@jeber-ibm
Copy link
Member Author

Looks like java.lang.Record is new in Java 14 -- https://docs.oracle.com/en/java/javase/14/docs/api/java.base/java/lang/Record.html. Not sure why my eclipse was complaining since compliance level was set to 1.7. I did this mostly as an exercise in using git.

@jeber-ibm jeber-ibm merged commit 2171d1c into IBM:main Mar 23, 2023
@jeber-ibm jeber-ibm deleted the TestFixes branch March 23, 2023 22:03
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.

2 participants