Skip to content

Conversation

@Achal1607
Copy link
Member

@Achal1607 Achal1607 commented Feb 17, 2025

Automate options available in OracleJDK download options in JDK Downloader.
Also, removed a test which was redundant. It was directly testing the stub instead of the original method due to which race condition is happening and it is failing.

@Achal1607 Achal1607 requested a review from sid-srini February 17, 2025 16:05
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 17, 2025
Copy link
Member

@sid-srini sid-srini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Achal1607 . This will significantly improve jdk access. One small suggestion would be to maintain the old jdk versions list as a fallback, in case the remote fetch fails due to some external cause or API change.

@oracle oracle deleted a comment from arvindaprameya Feb 18, 2025
@Achal1607 Achal1607 requested a review from sid-srini February 19, 2025 05:55
added fallback versions

removed redundant unit tests

updated JDK fallback versions
@Achal1607 Achal1607 force-pushed the automate-oracle-jdk-selection branch from e888244 to 75ac5ea Compare February 25, 2025 05:38
Copy link
Member

@sid-srini sid-srini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. LGTM :+1

@Achal1607 Achal1607 merged commit 8fa69e9 into oracle:main Feb 25, 2025
3 checks passed
@sid-srini sid-srini added this to the JVSC 24.0.0 milestone Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants