Bump Java to 21 (temurin) in CI workflows#805
Merged
Conversation
Co-authored-by: Reedyuk <3853006+Reedyuk@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update GitHub Actions to use Java 21 for firebase-tools compatibility
Bump Java to 21 (temurin) in CI workflows
Mar 13, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions CI environment to use a newer Java runtime for pull request testing.
Changes:
- Switch PR workflow JDK setup from Zulu 17 to Temurin 21.
- Update the shared composite “Setup Tests” action to use
actions/setup-java@v5and JDK 21.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/pull_request.yml | Updates the PR CI workflow to run matrix setup on Temurin JDK 21. |
| .github/actions/setup_test_action/action.yml | Updates the shared test setup composite action to use setup-java@v5 and Temurin JDK 21. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Comment on lines
26
to
31
| - name: Set up JDK | ||
| uses: actions/setup-java@v5 | ||
| with: | ||
| distribution: 'zulu' | ||
| java-version: '17' | ||
| distribution: 'temurin' | ||
| java-version: '21' | ||
| cache: gradle |
Comment on lines
7
to
12
| - name: Set up JDK | ||
| uses: actions/setup-java@v4 | ||
| uses: actions/setup-java@v5 | ||
| with: | ||
| distribution: 'zulu' | ||
| java-version: '17' | ||
| distribution: 'temurin' | ||
| java-version: '21' | ||
| cache: gradle |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
firebase-toolsnow requires JDK 21+, causing CI failures with the previously configured JDK 17.Changes
.github/actions/setup_test_action/action.yml— Switch fromzulu@17→temurin@21; bumpactions/setup-javav4 → v5.github/workflows/pull_request.yml— Same update for thejobMatrixSetupjob's JDK setup stepOriginal prompt
This pull request was created from Copilot chat.
💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.