Skip to content

Automate dropdown options of OracleJDK version in JDK downloader #376

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

Merged
merged 1 commit into from
Feb 25, 2025

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