-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 bytebuddy, asm and objenesis for full Java 11 support #1525
Conversation
cc @raphw |
gradle/dependencies.gradle
Outdated
@@ -20,9 +20,9 @@ libraries.bytebuddyandroid = "net.bytebuddy:byte-buddy-android:${versions.bytebu | |||
|
|||
libraries.errorprone = 'com.google.errorprone:error_prone_core:2.3.1' | |||
|
|||
libraries.objenesis = 'org.objenesis:objenesis:2.6' | |||
libraries.objenesis = 'org.objenesis:objenesis:3.0.1' |
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.
Version 3.0.1 requires Java 8. If this is an issue, we need to keep using the older version.
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.
I reverted this change.
Codecov Report
@@ Coverage Diff @@
## release/2.x #1525 +/- ##
==============================================
Coverage 88.59% 88.59%
Complexity 2401 2401
==============================================
Files 299 299
Lines 6039 6039
Branches 733 733
==============================================
Hits 5350 5350
Misses 510 510
Partials 179 179 Continue to review full report at Codecov.
|
Forgot to update ASM there, Byte Buddy bundles its own. LGTM! |
check list
including project members to get a better picture of the change
commit is meaningful and help the people that will explore a change in 2 years
Fixes #<issue number>
in the description if relevantFixes #<issue number>
if relevant