Skip to content

ci: Fix JDK 25 setup in Azure Pipelines CI - #1102

Merged
wenyt (wenytang-ms) merged 2 commits into
mainfrom
fix/ci-download-jdk-25
Sep 21, 2026
Merged

wenyt (wenytang-ms) merged 2 commits into
mainfrom
fix/ci-download-jdk-25

Conversation

@wenytang-ms

Copy link
Copy Markdown
Contributor

Summary

  • download Microsoft JDK 25 in both Azure Pipelines CI jobs
  • configure JavaToolInstaller to use the downloaded archive instead of the agent tool cache
  • verify the installed Java version before continuing

Validation

  • parsed .azure-pipelines/ci.yml with js-yaml
  • ran git diff --check

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The Linux job uses a Windows JDK archive and incompatible backslash paths.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity

Open (1)
What changed in this PR

Updates Azure Pipelines CI to download, install, and verify JDK 25.

Changes:

  • Downloads JDK 25 in both CI jobs.
  • Configures local archive installation.
  • Adds Java version checks.
File Summary
.azure-pipelines/​ci.yml Adds JDK setup steps to both CI jobs.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .azure-pipelines/ci.yml Outdated
@wenytang-ms wenyt (wenytang-ms) changed the title Fix JDK 25 setup in Azure Pipelines CI ci: Fix JDK 25 setup in Azure Pipelines CI Sep 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The Java checks must fail when the reported major version is not 25.

Review effort: Lite
Findings: None

Resolved since last review (1)

@wenytang-ms
wenyt (wenytang-ms) merged commit a47875d into main Sep 21, 2026
34 checks passed
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