Skip to content

Conversation

rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Oct 13, 2025

Fixed some Gradle warnings too. And removed OBE shadowing of okhttp as Spark 4 does not depend on okhttp.

Fixed some Gradle warnings too. And removed OBE shadowing of okhttp as Spark 4 does not depend on okhttp.
@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 14:45
Copy link
Contributor

@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

Updates third-party dependency versions and removes obsolete build configuration for Spark 4 compatibility. The changes focus on bumping logback from 1.5.18 to 1.5.19 and upgrading the MarkLogic Java Client from 8.0-SNAPSHOT to the stable 8.0.0 release.

  • Updated logback-classic dependency version to 1.5.19 across embedding model modules
  • Upgraded MarkLogic Java Client API from 8.0-SNAPSHOT to stable 8.0.0 release
  • Removed obsolete OkHttp shadowing configuration that is no longer needed with Spark 4

Reviewed Changes

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

File Description
flux-embedding-model-ollama/build.gradle Updated logback version and removed OkHttp relocation
flux-embedding-model-azure-open-ai/build.gradle Updated logback version to 1.5.19
flux-cli/build.gradle Upgraded Java Client API to 8.0.0 and updated related comments

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

Copy link

Copyright Validation Results
Total: 4 | Passed: 0 | Failed: 0 | Skipped: 4 | at: 2025-10-13 14:45:31 UTC | commit: 6f9c123

⏭️ Skipped (Excluded) Files

  • build.gradle
  • flux-cli/build.gradle
  • flux-embedding-model-azure-open-ai/build.gradle
  • flux-embedding-model-ollama/build.gradle

✅ All files have valid copyright headers!

@rjrudin rjrudin merged commit caea550 into develop Oct 13, 2025
3 checks passed
@rjrudin rjrudin deleted the feature/logback branch October 13, 2025 15:29
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