Skip to content

Conversation

rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Oct 9, 2025

Want to see the tests pass with this before releasing the Java Client.

Want to see the tests pass with this before releasing the Java Client.
@Copilot Copilot AI review requested due to automatic review settings October 9, 2025 12:28
Copy link

github-actions bot commented Oct 9, 2025

Copyright Validation Results
Total: 3 | Passed: 0 | Failed: 0 | Skipped: 3 | at: 2025-10-09 12:28:35 UTC | commit: 3f150c1

⏭️ Skipped (Excluded) Files

  • .copyrightconfig
  • NOTICE.txt
  • marklogic-spark-connector/build.gradle

✅ All files have valid copyright headers!

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 PR updates the Java Client library and related dependencies to their latest versions, upgrading from stable releases to snapshot/preview versions for testing purposes before releasing the Java Client.

  • Upgraded MarkLogic Java Client API from version 7.2.0 to 8.0-SNAPSHOT
  • Updated Jackson databind dependency from 2.18.2 to 2.19.0
  • Updated OkHttp dependency from 4.12.0 to 5.2.0

Reviewed Changes

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

File Description
marklogic-spark-connector/build.gradle Updates dependency versions for Jackson, MarkLogic Client API, and OkHttp
NOTICE.txt Updates third-party component versions and removes OkHttp entry from notices
.copyrightconfig Excludes .txt files from copyright checks

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

langchain4j 1.5.0 (Apache-2.0)
marklogic-client-api 7.2.0 (Apache-2.0)
okhttp 4.12.0 (Apache-2.0)
marklogic-client-api 8.0.0 (Apache-2.0)
Copy link

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

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

Version mismatch: The NOTICE.txt file shows marklogic-client-api version as 8.0.0, but the build.gradle file specifies 8.0-SNAPSHOT. These should match for consistency.

Copilot uses AI. Check for mistakes.

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@rjrudin rjrudin merged commit a40e821 into develop Oct 9, 2025
4 checks passed
@rjrudin rjrudin deleted the feature/bump-java-client branch October 9, 2025 12:45
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