Skip to content

Conversation

tcheeric
Copy link
Owner

Summary

  • Remove explicit Spring Framework versions from client module, letting Spring Boot 3.4.3 manage dependency versions.

Testing

  • mvn -q verify (fails: Could not find a valid Docker environment. Please see logs and check configuration)

Network Access

  • No external network requests were made; tests failed locally due to missing Docker environment.

https://chatgpt.com/codex/tasks/task_b_68991ba5b3b083319b9741963e7dee02

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

This PR removes explicit Spring Framework version management from the client module's POM file, allowing Spring Boot 3.4.3's BOM (Bill of Materials) to manage dependency versions automatically. This follows Spring Boot best practices for dependency management and ensures version consistency across Spring components.

  • Removed explicit version declarations for Spring dependencies
  • Deleted the spring-websocket.version property
  • Let Spring Boot BOM handle version resolution for spring-websocket, spring-retry, and spring-aspects

@@ -58,12 +56,10 @@
<dependency>
<groupId>org.springframework.retry</groupId>
<artifactId>spring-retry</artifactId>
Copy link
Preview

Copilot AI Aug 10, 2025

Choose a reason for hiding this comment

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

The spring-retry dependency is missing its version declaration after removing the explicit version, but the ${spring-retry.version} property is still referenced in the diff context. This will cause a build failure if the property is not defined elsewhere or if Spring Boot BOM doesn't manage this dependency.

Suggested change
<artifactId>spring-retry</artifactId>
<artifactId>spring-retry</artifactId>
<version>${spring-retry.version}</version>

Copilot uses AI. Check for mistakes.

@tcheeric tcheeric merged commit ac5b7c5 into develop Aug 10, 2025
1 check passed
@tcheeric tcheeric deleted the codex/update-spring-versions-in-pom.xml branch August 10, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant