Skip to content
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

docs(android): recommend JDK 17 as supported version #3598

Closed
wants to merge 1 commit into from
Closed

docs(android): recommend JDK 17 as supported version #3598

wants to merge 1 commit into from

Conversation

jpdriver
Copy link

JDK 11 is outdated.

I'd like to propose updating the guidance here to suggest using JDK 17 instead (latest LTS).

Prev. Discussions:-

  1. JDK 17 may not work for < 0.68 (see prev. discussion)
  2. JDK 18 should now work for 0.70+ now that Gradle 7.5 support has landed (see this issue)
    • but on macOS zulu18 is not available on Brew at time of writing
    • zulu17 is available
    • I figured suggesting JDK 17 everywhere might be simplest + best (?)

@netlify
Copy link

netlify bot commented Feb 28, 2023

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 97368ab
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/63fe7920961e9000082db502
😎 Deploy Preview https://deploy-preview-3598--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@cortinico
Copy link
Contributor

JDK 11 is outdated.

That's not correct. Specifically the distribution we're recommending (zulu from Azul) will receive support till 2026: https://www.azul.com/products/azul-support-roadmap/

We'll bump Java version to (most likely) 17 as soon as Android Gradle Plugin does the same and we align to that version inside React Native.

@cortinico cortinico closed this Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants