-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Switch to the Java 8 Date/Time API. #4556
Switch to the Java 8 Date/Time API. #4556
Conversation
Could you please make a separate PR for the multidex changes? I'm hoping it'll fix an issue with ACRA not finding classes in another dex. Edit: you also forgot to change NPE to your fork causing this PR to fail on CI |
Sure (although those changes were made because API desugaring requires multidex for all build types).
Thanks! |
30c9649
to
c1cdb10
Compare
c219aee
to
a88b564
Compare
5bb9e4f
to
aa87082
Compare
266f3ea
to
f56cdf8
Compare
Closing and reopening to trigger a rebuild. |
f56cdf8
to
742cad0
Compare
b3ab770
to
414e421
Compare
414e421
to
c745b84
Compare
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.
Thank you for improving the codebase! I tested thoroughly the APK you provided, trying multiple actions with dates, amd everything worked fine.
What is it?
Description of the changes in your PR
Relies on the following changes
APK testing
debug.zip
Due diligence