Skip to content

chore : Added the JDK setup to the rl-scanner job - #1138

Merged
subhankarmaiti merged 1 commit into
mainfrom
chore/rl-pipeline-fix
Nov 12, 2026
Merged

subhankarmaiti merged 1 commit into
mainfrom
chore/rl-pipeline-fix

Conversation

@subhankarmaiti

@subhankarmaiti subhankarmaiti commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Changes

The rl-scanner job in .github/workflows/release.yml had no JDK setup, so Gradle ran on the runner's default Java 25. Gradle 8.11.1 can't run on that and fails with Unsupported class file major version 69.

This adds the shared ./.github/actions/setup action (Temurin JDK 17 + Gradle) to the job, before ./gradlew :auth0:assembleRelease, same as test.yml. The release job is unchanged since its maven-publish action already pins JDK 17.

Same change as #1133, for main. No SDK code or public API changes.

References

Checklist

Summary by CodeRabbit

  • Chores
    • Updated the release workflow to run its local setup step before building with Gradle. This changes the order of CI preparation and build steps; no end-user-facing features or behavior changes are included in this update.

@subhankarmaiti
subhankarmaiti requested a review from a team as a code owner November 12, 2026 10:42
@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The rl-scanner job in the release workflow now runs the local setup action before the Gradle build.

Changes: Release CI Setup

Layer / File(s) Summary
Configure the release job
.github/workflows/release.yml
The rl-scanner job runs ./.github/actions/setup before the Gradle build.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Suggested reviewers: pmathew92

Merge Risk: Minimal · up to c5db6

This change fixes the release scanner job so it builds with JDK 17 instead of the runner's default Java. It touches no SDK code or public API, and no merge-blocking risk remains.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title names a JDK setup change for the rl-scanner job, which matches the single workflow change (the job now runs the local setup action before the Gradle build). The title uses the generic 'chore…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title names a JDK setup change for the rl-scanner job, which matches the single workflow change (the job now runs the local setup action before the Gradle build). The title uses the generic 'chore :' prefix, but it still names the primary change. It is accurate and specific enough for a reviewer scanning history.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Team
  • Run ID: ba212b5e-5b4e-4ed7-86a2-c6cde84f473b
Commits

Reviewing files that changed from the base of the PR and between 32f424c and c5db6cd.

Files selected for processing: 1

  • .github/workflows/release.yml

Included review availability: This review used your included allowance. 6 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 8 reviews per hour.

@subhankarmaiti
subhankarmaiti merged commit e47a266 into main Nov 12, 2026
9 checks passed
@subhankarmaiti
subhankarmaiti deleted the chore/rl-pipeline-fix branch November 12, 2026 11:06
@pmathew92 pmathew92 mentioned this pull request Nov 13, 2026
3 tasks done
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