Skip to content

Feature/update to java 21#403

Open
Naenyn wants to merge 3 commits intouPortal-Project:masterfrom
Naenyn:feature/update-to-java-21
Open

Feature/update to java 21#403
Naenyn wants to merge 3 commits intouPortal-Project:masterfrom
Naenyn:feature/update-to-java-21

Conversation

@Naenyn
Copy link
Contributor

@Naenyn Naenyn commented May 5, 2025

No description provided.

@Naenyn Naenyn requested review from bjagg and Copilot May 5, 2025 17:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the CI workflow configuration to support testing on Java 21 by updating the Java version matrix and adding a manual workflow trigger.

  • Added the workflow_dispatch trigger to allow manual CI runs.
  • Updated the Java version matrix to include Java 17 and 21.
  • Changed the checkout action version from v4 to v3.
Comments suppressed due to low confidence (1)

.github/workflows/CI.yml:55

  • Reverting the checkout action from v4 to v3 might introduce unintended issues; please confirm that this downgrade is intentional and does not impact the CI reliability.
uses: actions/checkout@v3

@Naenyn Naenyn requested a review from Copilot May 5, 2025 19:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the CI workflow configuration to support Java 21 and streamline the JDK setup.

  • Added a manual trigger via workflow_dispatch
  • Updated the matrix configuration to include Java versions 17 and 21, replacing previous Java 8/11 versions
  • Adjusted job steps to reflect the new JDK version configuration

Copy link
Member

@bjagg bjagg left a comment

Choose a reason for hiding this comment

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

Hold off on merging until we release v5.10.1 for uPortal 5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants