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
chore(deps): Amplify Android 2.15.0 #4279
Conversation
This reverts commit 1c24861.
packages/amplify_datastore/android/gradle/wrapper/gradle-wrapper.properties
Show resolved
Hide resolved
...ons/push/amplify_push_notifications/example/android/gradle/wrapper/gradle-wrapper.properties
Show resolved
Hide resolved
@@ -45,7 +45,7 @@ jobs: | |||
- uses: actions/setup-java@cd89f46ac9d01407894225f350157564c9c7cee2 # 3.12.0 | |||
with: | |||
distribution: "corretto" # Amazon Corretto Build of OpenJDK | |||
java-version: "11" | |||
java-version: "17" |
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.
why to update the java version? I think the java version used in our android packages are still version 8 and so not sure why to use different version on CI.
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.
The newer gradle version to 8.1.0 requires java 17 and above to run.
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 see, so I think we should update the java version and jvmTraget in the android packages as well
uses: actions/setup-java@cd89f46ac9d01407894225f350157564c9c7cee2 # 3.12.0 | ||
with: | ||
distribution: "corretto" # Amazon Corretto Build of OpenJDK | ||
java-version: "17" |
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.
same as above
...ndroid/src/test/kotlin/com/amazonaws/amplify/amplify_datastore/AmplifyDataStorePluginTest.kt
Outdated
Show resolved
Hide resolved
This reverts commit 01d87aa.
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.
Great job, 🚢 -it!
* chore(deps): Amplify Android 2.15.0 * chore: update Kotlin version --------- Co-authored-by: Nika Hassani <nikaws@amazon.com> Co-authored-by: Jordan Nelson <Jordanryannelson@gmail.com>
* chore(deps): Amplify Android 2.15.0 * chore: update Kotlin version --------- Co-authored-by: Nika Hassani <nikaws@amazon.com> Co-authored-by: Jordan Nelson <Jordanryannelson@gmail.com>
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.