Skip to content

Conversation

rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Oct 1, 2025

No warnings or errors locally

@rjrudin rjrudin requested a review from anu3990 as a code owner October 1, 2025 17:33
@Copilot Copilot AI review requested due to automatic review settings October 1, 2025 17:33
Copy link

@Copilot Copilot AI left a 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 pull request upgrades the project from Gradle 8.14.3 to Gradle 9.1.0 and removes unnecessary classpath configurations from the Gradle wrapper scripts.

  • Updates Gradle version to 9.1.0 in wrapper properties
  • Removes obsolete classpath variable usage from both Unix and Windows wrapper scripts
  • Simplifies copyright notice in the Unix wrapper script

Reviewed Changes

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

File Description
gradle/wrapper/gradle-wrapper.properties Updates Gradle distribution URL to version 9.1.0
gradlew Removes classpath variable usage and simplifies copyright notice
gradlew.bat Removes classpath variable usage from Windows wrapper script

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

github-actions bot commented Oct 1, 2025

Copyright Validation Results
Total: 8 | Passed: 0 | Failed: 0 | Skipped: 8 | at: 2025-10-01 18:06:46 UTC | commit: 28179ac

⏭️ Skipped (Excluded) Files

  • Jenkinsfile
  • build.gradle
  • gradle/wrapper/gradle-wrapper.jar
  • gradle/wrapper/gradle-wrapper.properties
  • gradlew
  • gradlew.bat
  • marklogic-client-api-functionaltests/build.gradle
  • marklogic-client-api/build.gradle

✅ All files have valid copyright headers!

BillFarber
BillFarber previously approved these changes Oct 1, 2025
No warnings or errors locally.

Also shifting to the 11.3.2 build temporarily to try things out.

And fixing a problem with the ml-development-tools tests.
@rjrudin rjrudin merged commit d8f1519 into develop Oct 1, 2025
3 checks passed
@rjrudin rjrudin deleted the feature/gradle9 branch October 1, 2025 18:21
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