-
Notifications
You must be signed in to change notification settings - Fork 2
PR for v1.2.7 #537
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
ThomasJejkal
wants to merge
50
commits into
main
Choose a base branch
from
development
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
PR for v1.2.7 #537
Changes from 36 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
3695b58
Update dependency org.springframework.boot:spring-boot-dependencies t…
renovate[bot] 80c80bf
Update dependency org.springframework.boot:spring-boot-starter-actuat…
renovate[bot] 7aaf314
Merge pull request #536 from kit-data-manager/renovate/org.springfram…
ThomasJejkal 56403b5
Merge pull request #535 from kit-data-manager/renovate/springbootversion
ThomasJejkal fcc5dcb
Update dependency gradle to v9.2.0
renovate[bot] bf9368b
Update dependency com.fasterxml.jackson.datatype:jackson-datatype-jsr…
renovate[bot] 9b449fa
Update dependency com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-prov…
renovate[bot] bc5479b
Update dependency com.fasterxml.jackson.module:jackson-module-afterbu…
renovate[bot] f6fa21b
Update dependency de.codecentric:spring-boot-admin-starter-client to …
renovate[bot] 3255005
Update springDocVersion to v2.8.14
renovate[bot] 6c1af80
Merge pull request #544 from kit-data-manager/renovate/springdocversion
ThomasJejkal d2602a5
Merge pull request #543 from kit-data-manager/renovate/de.codecentric…
ThomasJejkal 691bd96
Merge pull request #542 from kit-data-manager/renovate/com.fasterxml.…
ThomasJejkal 04d2121
Merge branch 'development' into renovate/com.fasterxml.jackson.jaxrs-…
ThomasJejkal d9a505f
Merge pull request #541 from kit-data-manager/renovate/com.fasterxml.…
ThomasJejkal 17fcf5a
Update dependency com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-prov…
renovate[bot] 1f1416b
Merge branch 'development' into renovate/com.fasterxml.jackson.dataty…
ThomasJejkal b780090
Merge pull request #540 from kit-data-manager/renovate/com.fasterxml.…
ThomasJejkal 5fc0303
Merge pull request #545 from kit-data-manager/renovate/com.fasterxml.…
ThomasJejkal 655aec9
Update dependency com.fasterxml.jackson.datatype:jackson-datatype-jod…
renovate[bot] ec18780
Merge pull request #539 from kit-data-manager/renovate/com.fasterxml.…
ThomasJejkal b017211
Merge pull request #538 from kit-data-manager/renovate/gradle-9.x
ThomasJejkal 15d3205
Update plugin io.freefair.lombok to v9.1.0
renovate[bot] e5d6e64
Update plugin io.freefair.maven-publish-java to v9.1.0
renovate[bot] 766528c
Update dependency commons-io:commons-io to v2.21.0
renovate[bot] f594bf2
Merge pull request #548 from kit-data-manager/renovate/commons-io-com…
ThomasJejkal 95c1775
Merge pull request #547 from kit-data-manager/renovate/io.freefair.ma…
ThomasJejkal 9621207
Merge branch 'development' into renovate/io.freefair.lombok-9.x
ThomasJejkal b2501d0
Merge pull request #546 from kit-data-manager/renovate/io.freefair.lo…
ThomasJejkal c8e6950
Update plugin org.owasp.dependencycheck to v12.1.9
renovate[bot] 8fdca4e
Update dependency org.springframework:spring-messaging to v6.2.13
renovate[bot] e95eab5
Update dependency org.springframework:spring-messaging to v7
renovate[bot] a2d7a3f
Merge pull request #552 from kit-data-manager/renovate/org.springfram…
ThomasJejkal 8b94bdf
Merge branch 'development' into renovate/org.springframework-spring-m…
ThomasJejkal 8a0d0ea
Merge pull request #551 from kit-data-manager/renovate/org.springfram…
ThomasJejkal e97d322
Merge pull request #550 from kit-data-manager/renovate/org.owasp.depe…
ThomasJejkal ab19ba2
Update dependency org.springframework.data:spring-data-elasticsearch …
renovate[bot] 1c54ce3
Update dependency gradle to v9.2.1
renovate[bot] 82569f6
Update dependency org.springframework.restdocs:spring-restdocs-mockmv…
renovate[bot] 1663b1a
Update dependency org.springframework:spring-messaging to v7.0.1
renovate[bot] 0c0dda7
Update dependency org.springframework.boot:spring-boot-dependencies t…
renovate[bot] d769aea
Update dependency org.springframework.boot:spring-boot-starter-actuat…
renovate[bot] b19255e
Update actions/checkout action to v6
renovate[bot] 5826c32
Merge pull request #553 from kit-data-manager/renovate/org.springfram…
VolkerHartmann af3dc5b
Merge pull request #555 from kit-data-manager/renovate/gradle-9.x
VolkerHartmann ea4bf90
Merge pull request #556 from kit-data-manager/renovate/org.springfram…
VolkerHartmann aab2a94
Merge pull request #557 from kit-data-manager/renovate/org.springfram…
VolkerHartmann 8244b12
Merge pull request #558 from kit-data-manager/renovate/springbootversion
VolkerHartmann 3a1c862
Merge pull request #559 from kit-data-manager/renovate/org.springfram…
VolkerHartmann a7b7400
Merge pull request #560 from kit-data-manager/renovate/actions-checko…
VolkerHartmann File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
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.
🧩 Analysis chain
Updating
org.springframework:spring-messagingfrom6.2.12to7.0.0is a major version change, not a routine patch or minor bump. Spring 7.0 introduces significant changes from the 6.x series, including API modifications and potential breaking changes.Before merging, verify:
Run the verification script below to check for deprecated or removed Spring messaging APIs:
🏁 Script executed:
Length of output: 53
🏁 Script executed:
Length of output: 74000
🏁 Script executed:
Length of output: 412
🌐 Web query:
💡 Result:
Key breaking-change items in Spring Framework 7.0 that affect messaging (spring-messaging / JMS):
Recommended quick steps when migrating messaging code:
Sources: Spring Framework 7.0 release notes and announcement, Maven/Central spring-messaging listings, 7.0 Javadoc. [1][2][3][4]
References
[1] Spring Framework 7.0 Release Notes (spring-projects/spring-framework wiki).
[2] Spring blog: Spring Framework 7.0.0-M7 announcement.
[3] Maven Central / spring-messaging 7.0 listings.
[4] spring-messaging 7.0.0 Javadoc.
🏁 Script executed:
Length of output: 295
🌐 Web query:
💡 Result:
Key breaking changes when migrating to Spring Framework 7.0 (focused on the Jakarta package migration) — short checklist and action items:
What changed
Practical migration steps
Tools & references for automated help
Primary sources
If you want, I can:
🏁 Script executed:
Length of output: 8202
🏁 Script executed:
Length of output: 403
Based on the critical findings from build.gradle analysis, I can now generate the rewritten review:
The explicit dependency on
org.springframework:spring-messaging:7.0.0conflicts with Spring Boot 3.5.7, which uses Spring Framework 6.2.x. This creates a version mismatch where incompatible versions of the Spring framework are forced together on the classpath, likely causing:Fix required before merge:
Either:
6.2.12or compatible 6.2.x version)Do not merge with this version mismatch. Verify your intended target Spring/Spring Boot version and ensure all Spring framework dependencies align.
🌐 Web query:
💡 Result:
Spring Boot 3.5.7 is built against (uses) Spring Framework 6.2.12. [1][2]
Sources:
Fix version incompatibility: Spring messaging 7.0.0 cannot be used with Spring Boot 3.5.7.
Spring Boot 3.5.7 uses Spring Framework 6.2.12 transitively. The explicit dependency on
org.springframework:spring-messaging:7.0.0creates a major version conflict—Spring 7.0 introduces breaking changes incompatible with the 6.2.x components already on the classpath, causing API mismatches and runtime failures.Required resolution:
Choose one:
spring-messagingto6.2.12(matching Spring Boot 3.5.7's baseline)Do not merge until versions are aligned.
🤖 Prompt for AI Agents