GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Upgrade to Gradle 9.2.0 by Copilot · Pull Request #4171 · graphql-java/graphql-java

Conversation

Copilot AI commented Nov 16, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Upgrading graphql-java from Gradle 8.13 to Gradle 9.2.0

Plan

  • Update gradle-wrapper.properties to use Gradle 9.2.0
  • Update bnd plugin from 6.4.0 to 7.1.0 (required for Gradle 9 compatibility)
  • Fix deprecated Convention API usage in bnd configuration
  • Fix settings.gradle URL assignment syntax (Gradle 9 requirement)
  • Add explicit task dependencies for generateTestGrammarSource and generateJmhGrammarSource
  • Add JUnit Platform launcher dependency (required for Gradle 9)
  • Run build to verify compatibility
  • Run tests to ensure everything works
  • Fix deprecation warnings (multi-string dependencies, archives configuration, property assignments)
  • Update Gradle wrapper jar and scripts (gradle-wrapper.jar, gradlew, gradlew.bat)
  • Final verification

Changes Made

  • Updated Gradle wrapper to version 9.2.0 (properties, jar, and scripts)
  • Updated biz.aQute.bnd.builder plugin from 6.4.0 to 7.1.0
  • Removed deprecated Convention API usage and replaced with proper BundleTaskExtension
  • Fixed settings.gradle to use assignment syntax for maven repository URL
  • Added explicit task ordering with mustRunAfter for grammar generation tasks
  • Added org.junit.platform:junit-platform-launcher as testRuntimeOnly dependency
  • Fixed multi-string dependency notation for junit (changed to single-string)
  • Removed deprecated archives configuration
  • Fixed property assignment syntax (version, required)
  • Fixed Task.project usage in doLast blocks
  • Updated Gradle wrapper jar and scripts using ./gradlew wrapper --gradle-version=9.2.0
  • Build and tests work successfully with Gradle 9.2.0

Notes

  • This upgrade unblocks issue Build and Test using Java 25 #4119 as mentioned in the problem statement
  • One remaining deprecation warning from the bnd plugin itself (Task.project during jar task execution) is outside our control
  • All user-defined deprecations have been fixed
  • Gradle wrapper files (jar and scripts) have been properly updated to 9.2.0
Original prompt

This section details on the original issue you should resolve

<issue_title>Upgrade to Gradle 9</issue_title>
<issue_description>Beyond accessing the latest Gradle features and bug fixes, it unblocks #4119</issue_description>

Comments on the Issue (you are @copilot in this section)

@andimarek In general we want to upgrade to gradle 9.

Sometimes we are blocked by plugins we use, so we must probably upgrade dependencies first </comment_new>

  • Fixes Upgrade to Gradle 9 #4170

💡 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.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Initial plan
6dea59e
Copilot AI assigned Copilot and andimarek Nov 16, 2025
Copilot started work on behalf of andimarek November 16, 2025 03:40
View session
@andimarek

andimarek commented Nov 16, 2025

Copy link
Copy Markdown
Member

Make sure to upgrade the gradle wrapper itself too once upgraded to gradle 9
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Copilot AI and others added 2 commits November 16, 2025 04:06
@andimarek
Upgrade to Gradle 9.2.0 with plugin updates and fixes
4db3cc4 
Co-authored-by: andimarek <[email protected]>
@andimarek
Fix deprecation warnings for Gradle 9 compatibility
3e1dd93 
Co-authored-by: andimarek <[email protected]>
Copilot AI changed the title [WIP] Upgrade project to Gradle 9 Upgrade to Gradle 9.2.0 Nov 16, 2025
Copilot AI requested a review from andimarek November 16, 2025 04:12
Copilot finished work on behalf of andimarek November 16, 2025 04:12
@github-actions

github-actions Bot commented Nov 16, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Test Results

  326 files    326 suites   5m 4s ⏱️
5 218 tests 5 209 ✅ 9 💤 0 ❌
5 307 runs  5 298 ✅ 9 💤 0 ❌

Results for commit 10e3596.

♻️ This comment has been updated with latest results.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@andimarek

andimarek commented Nov 16, 2025

Copy link
Copy Markdown
Member

@copilot Make sure to update the gradle wrapper itself too
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Copilot started work on behalf of andimarek November 16, 2025 04:13
View session
@andimarek
Update Gradle wrapper jar and scripts to version 9.2.0
10e3596 
Co-authored-by: andimarek <[email protected]>

Copilot AI commented Nov 16, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Make sure to upgrade the gradle wrapper itself too once upgraded to gradle 9

Done in commit 10e3596. Updated gradle-wrapper.jar, gradlew, and gradlew.bat using ./gradlew wrapper --gradle-version=9.2.0.

@copilot Make sure to update the gradle wrapper itself too

Done in commit 10e3596. The gradle wrapper jar and scripts have been updated to 9.2.0.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Copilot finished work on behalf of andimarek November 16, 2025 04:19
andimarek
andimarek approved these changes Nov 16, 2025
View reviewed changes
@andimarek
andimarek marked this pull request as ready for review November 16, 2025 04:32
@andimarek
andimarek requested a review from Copilot November 16, 2025 04:32
Copilot started reviewing on behalf of andimarek November 16, 2025 04:33
View session
Copilot finished reviewing on behalf of andimarek November 16, 2025 04:34
Copilot AI reviewed Nov 16, 2025
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

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 upgrades graphql-java from Gradle 8.13 to Gradle 9.2.0, addressing compatibility issues and deprecation warnings introduced in the newer Gradle version.

Key changes:

  • Updated Gradle wrapper and bnd plugin to versions compatible with Gradle 9
  • Migrated from deprecated Convention API to BundleTaskExtension for OSGi bundle configuration
  • Fixed syntax and API usage patterns deprecated in Gradle 9

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
gradle/wrapper/gradle-wrapper.properties Updated distribution URL to Gradle 9.2.0
gradle/wrapper/gradle-wrapper.jar Updated wrapper JAR binary to Gradle 9.2.0
gradlew Updated wrapper script with new invocation syntax for Gradle 9
gradlew.bat Updated Windows wrapper script with new invocation syntax for Gradle 9
settings.gradle Fixed maven repository URL syntax to use assignment operator
build.gradle Updated bnd plugin version, migrated deprecated Convention API, added JUnit Platform launcher dependency, fixed deprecated property assignments, and added explicit task dependencies

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@andimarek
andimarek merged commit 1846e70 into master Nov 16, 2025
11 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@andimarek andimarek mentioned this pull request Nov 16, 2025
Upgrade to Gradle 9 #4170
Closed
@reneleonhardt

reneleonhardt commented Nov 16, 2025

Copy link
Copy Markdown

Why is JUnit 4 still needed if 5 is already required?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
This was referenced Nov 10, 2026
OSGi manifest headers missing from published JARs since 26.0 #4547
Open
Restore OSGi manifest headers in the published jar #4548
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
+1 more reviewer
@andimarek andimarek andimarek approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@andimarek andimarek

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Upgrade to Gradle 9

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@andimarek @reneleonhardt
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.