Skip to content

Update build.yml add distribution: 'temurin' of JDK - #600

Merged
wenyt (wenytang-ms) merged 5 commits into
microsoft:mainfrom
mozhuanzuojing:patch-2
Nov 10, 2025
Merged

Update build.yml add distribution: 'temurin' of JDK#600
wenyt (wenytang-ms) merged 5 commits into
microsoft:mainfrom
mozhuanzuojing:patch-2

Conversation

@mozhuanzuojing

Copy link
Copy Markdown
Contributor

add distribution: 'temurin' of JavaSE-21

add distribution: 'temurin' of JavaSE-21
@chagong

Copy link
Copy Markdown
Contributor

mozhuanzuojing can you explain why it needs to use termurin distribution?

@mozhuanzuojing

Copy link
Copy Markdown
ContributorAuthor

mozhuanzuojing can you explain why it needs to use termurin distribution?

I see. The eclipse.jdt.ls project is configured that way, and I just want to maintain consistency. I don't have too many thoughts about it.

@wenytang-ms

Copy link
Copy Markdown
Contributor

mozhuanzuojing please also upgrade the action version to v5, thanks!

@mozhuanzuojing

Copy link
Copy Markdown
ContributorAuthor

mozhuanzuojing please also upgrade the action version to v5, thanks!

Updated to v5

@wenytang-ms
wenyt (wenytang-ms) merged commit dabcc49 into microsoft:mainNov 10, 2025
4 checks passed
Sign up for freeto 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

@mozhuanzuojing@chagong@wenytang-ms