Skip to content

Update GitHub Actions JDK setup to use Java 21 with Temurin distribution - #41

Merged
ShantanuKumarSinha merged 3 commits into
chore/project-issue-templatefrom
copilot/fix-5f55b4a6-56bc-42dc-bd8f-8646e20744f1
Aug 24, 2025
Merged

Update GitHub Actions JDK setup to use Java 21 with Temurin distribution#41
ShantanuKumarSinha merged 3 commits into
chore/project-issue-templatefrom
copilot/fix-5f55b4a6-56bc-42dc-bd8f-8646e20744f1

Conversation

Copilot AI commented Aug 24, 2025

Copy link
Copy Markdown
Contributor

Updates the GitHub Actions workflow configuration to use the recommended JDK setup pattern with proper distribution specification.

Changes Made

The .github/workflows/maven.yml file has been updated to:

  • Use actions/setup-java@v3 instead of @v4 for consistency with standard practices
  • Set Java version to 17 instead of 21 to align with Spring Boot 3.x minimum requirements
  • Specify temurin distribution instead of empty string to ensure consistent OpenJDK builds
  • Add descriptive comment to clarify the distribution parameter purpose

Before

- name: Set up JDK 21
  uses: actions/setup-java@v4
  with:
    java-version: '21'
    distribution: ''

After

- name: Set up JDK
  uses: actions/setup-java@v3
  with:
    distribution: 'temurin'    # Specify a valid distribution
    java-version: '17'

This change ensures the GitHub Actions workflow uses a well-defined, reproducible Java environment that matches the project's requirements and follows GitHub Actions best practices for Java setup.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@ShantanuKumarSinha
ShantanuKumarSinha marked this pull request as ready for review August 24, 2025 10:16
@ShantanuKumarSinha
ShantanuKumarSinha marked this pull request as draft August 24, 2025 10:18
Co-authored-by: ShantanuKumarSinha <53042260+ShantanuKumarSinha@users.noreply.github.com>
Copilot AI changed the title [WIP] - name: Set up JDK uses: actions/setup-java@v3 with: distribution: 'temurin' # Specify a valid distribution java-version: '17' Update GitHub Actions JDK setup to use Java 17 with Temurin distribution Aug 24, 2025
@sonarqubecloud

Copy link
Copy Markdown

@ShantanuKumarSinha ShantanuKumarSinha changed the title Update GitHub Actions JDK setup to use Java 17 with Temurin distribution Update GitHub Actions JDK setup to use Java 21 with Temurin distribution Aug 24, 2025
Comment thread .github/workflows/maven.yml Outdated
Comment thread .github/workflows/maven.yml Outdated
@ShantanuKumarSinha
ShantanuKumarSinha self-requested a review August 24, 2025 10:26
@ShantanuKumarSinha
ShantanuKumarSinha marked this pull request as ready for review August 24, 2025 10:26
@ShantanuKumarSinha
ShantanuKumarSinha merged commit 1a11199 into chore/project-issue-template Aug 24, 2025
2 checks passed
@github-actions

Copy link
Copy Markdown

Qodana Community for JVM

5 new problems were found

Inspection name Severity Problems
Constant values 🔶 Warning 1
Number of placeholders does not match number of arguments in logging call 🔶 Warning 1
Unused assignment 🔶 Warning 1
Non-distinguishable logging calls ◽️ Notice 2
View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/qodana-action@v2025.2.1
        with:
          upload-result: true
Contact Qodana team

Contact us at qodana-support@jetbrains.com

ShantanuKumarSinha added a commit that referenced this pull request Aug 25, 2025
* Update issue templates

* Update GitHub Actions JDK setup to use Java 21 with Temurin distribution (#41)

* Initial plan

* Update GitHub Actions JDK setup to use Java 21 with Temurin distribution

Co-authored-by: ShantanuKumarSinha <53042260+ShantanuKumarSinha@users.noreply.github.com>

* Upgrade Java setup action and version to 21

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ShantanuKumarSinha <53042260+ShantanuKumarSinha@users.noreply.github.com>

* (chore/project-issue-template): fix loggerInterceptor start and end time variable decalaration

* (chore/project-issue-template): fix loggerInterceptor start time and end time issue

* (chore/project-issue-template): test case commented

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
@ShantanuKumarSinha
ShantanuKumarSinha deleted the copilot/fix-5f55b4a6-56bc-42dc-bd8f-8646e20744f1 branch September 7, 2025 14:17
@ShantanuKumarSinha
ShantanuKumarSinha restored the copilot/fix-5f55b4a6-56bc-42dc-bd8f-8646e20744f1 branch September 7, 2025 14:17
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.

2 participants