Closed ParaskP7 closed 9 months ago
Awesome @oguzkocer , thanks for adding JDK17 support to this repo, then reviewing and tentatively approving the changes, pending some extra testing! 🙇 ❤️ 🚀
FYI: I went ahead, marked this PR as Ready for review
and removed the [PR] Not Ready for Merge
label. The last thing for us to do is to coordinate the merge wit with all the other library and client PRs, merging this PR only when every other such PR is ready to be merged as well.
Thanks for updating the publish-to-s3
Gradle plugin @oguzkocer , LGTM, you rock! 🙇 ❤️ 🚀
Platform Request:
pdnsEh-13V-p2
Project Thread:paaHJt-57Z-p2
This PR upgraded Gradle to 8.2.1 and AGP to 8.0.2.
FYI: The 8.1.1 Gradle upgrades had to happen in 2 parts (67e1bd02d91c5b892434b53b7181ea1d57f10af2 and 4f080b069ff6dcb7549ce49ce8c42f808ef57e56). This is because of the breaking changes that this upgrade introduces, including the
AGP
upgrade and its associated breaking changes (namespace, build option, etc).🚫 Blocked By:
Android Gradle plugin requires Java 17 to run. You are currently using Java 11.
CI Failure
``` [2023-07-11T11:31:30Z] FAILURE: Build failed with an exception. -- | [2023-07-11T11:31:30Z] | [2023-07-11T11:31:30Z] * Where: | [2023-07-11T11:31:30Z] Build file '/var/lib/buildkite-agent/builds/ci-android-i-0c37688b1a5352a5d-1/automattic/wordpress-login-flow-android/WordPressLoginFlow/build.gradle' line: 2 | [2023-07-11T11:31:30Z] | [2023-07-11T11:31:30Z] * What went wrong: | [2023-07-11T11:31:30Z] An exception occurred applying plugin request [id: 'com.android.library', version: '8.0.2'] | [2023-07-11T11:31:30Z] > Failed to apply plugin 'com.android.internal.library'. | [2023-07-11T11:31:30Z] > Android Gradle plugin requires Java 17 to run. You are currently using Java 11. | [2023-07-11T11:31:30Z] Your current JDK is located in /usr/lib/jvm/java-11-openjdk-11.0.19.0.7-1.amzn2.0.1.x86_64 | [2023-07-11T11:31:30Z] You can try some of the following options: | [2023-07-11T11:31:30Z] - changing the IDE settings. | [2023-07-11T11:31:30Z] - changing the JAVA_HOME environment variable. | [2023-07-11T11:31:30Z] - changing `org.gradle.java.home` in `gradle.properties`. ```Breaking Change List:
Associated clients
You can reference the
[Build] Upgrade Gradle to 8.1.1 and AGP to 8.0.2
PRs of the below associated clients for testing/verification purposes:To test
Merge instructions
Ready for review
.[PR] Not Ready for Merge
label.