Skip to content

Use older Piston version for Java 21 compatibility#3603

Open
SirYwell wants to merge 1 commit into
mainfrom
fix/piston-j21
Open

Use older Piston version for Java 21 compatibility#3603
SirYwell wants to merge 1 commit into
mainfrom
fix/piston-j21

Conversation

@SirYwell

Copy link
Copy Markdown
Member

Overview

Description

To still support Java 21, we need to use an older Piston version.
From a quick look, it doesn't seem like we're missing out on anything important.

Submitter Checklist

  • Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
  • Ensure that the pull request title represents the desired changelog entry.
  • New public fields and methods are annotated with @since TODO.
  • I read and followed the contribution guidelines.

@SirYwell
SirYwell requested a review from a team as a code owner July 24, 2026 07:25
Copilot AI review requested due to automatic review settings July 24, 2026 07:25

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.

Pull request overview

This PR updates the Gradle version catalog to keep the project compatible with Java 21 by downgrading the EngineHub Piston dependency to a version that still supports Java 21 at runtime.

Changes:

  • Downgrade piston from 0.6.0 to 0.5.11 in the version catalog.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MattBDev

Copy link
Copy Markdown
Contributor

While I'm okay with downgrading for now. I hope we can put a timeline on moving to Java 25.

@PierreSchwang PierreSchwang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could potentially add to dependabot exclusion-list

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.

4 participants