-
Notifications
You must be signed in to change notification settings - Fork 165
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
Update maven config and tests to allow Java 8, 11 and 17 builds #653
Conversation
dotasek
commented
Nov 10, 2021
- Updates several dependency versions.
- Conditionally sets up surefire tests with an optional argument that allows Java 8 surefire tests to run without encountering garbage collection errors.
- Updates the JAVA_HOME resolution for Mac in UtilityTests to use the resolved path instead of a string, which was failing on different Mac setups due to inconsistencies in using a trailing forward slash for the value.
|
Codecov Report
@@ Coverage Diff @@
## master #653 +/- ##
=======================================
Coverage 10.03% 10.03%
=======================================
Files 1728 1728
Lines 538009 538009
Branches 159335 159340 +5
=======================================
+ Hits 53966 53968 +2
Misses 464364 464364
+ Partials 19679 19677 -2
Continue to review full report at Codecov.
|
No longer Work in Progress; ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff.