feat : upgrade to spring boot 4#2333
Merged
Merged
Conversation
Contributor
rajadilipkolli
commented
May 5, 2026
rajadilipkolli
commented
- Chores
- Upgraded Maven Wrapper to v3.3.4 and standardized projects on Apache Maven 3.9.15; legacy wrapper downloader removed across modules.
- Rewrote wrapper scripts to download/extract/cache distributions with optional SHA‑256 verification, improved platform handling and auth fallbacks.
- Updated POMs to enable annotation processing (Lombok) and modernized test dependencies and Testcontainers artifacts.
- Adjusted devcontainer dind parameters and expanded VS Code extension recommendations.
- Aligned tests/configs to new test slice packages and updated some MongoDB property keys and logging formats.
* feat : upgrade to spring boot 4 * Adds more extensions * upgrade maven wrapper * Upgrade to spring boot * fix issue with compilation * Upgrade to SB4 * fix failures * fix more * wrapper fix * fix : issues with test case failures * upgrade * upgrade and fix * fix : failures * typo fix * upgrade to sb4 * Upgrade to SB4 * upgrade * upgrade modules * more modules * migrate more * last one for the day * Upgrade to SB4 * Upgrade to SB4 * Upgrade * upgrade * upgrade * upgrade * upgrade * upgrade * fix : compilation and build failure issue * upgrade * upgrade * ugrade client * revert and fix * fix : compilation issues * revert back
ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one. |
Contributor
Author
|
@hantsy , Can you please retrigger all 3 failed workflows. It was working for me. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates various Spring Boot projects to version 4.0.6, upgrades Kotlin to 2.2.21, and modernizes the Maven wrapper scripts. It also includes several test-related improvements, such as replacing deprecated test annotations and configuring Testcontainers. My review identified a few areas for improvement: the manual logger instantiation in DataInitializer should be replaced with the @Slf4j annotation, and raw types in Testcontainers definitions should be replaced with generic types to ensure type safety.
rajadilipkolli
changed the title
* feat : adds workflow for kafka and rabbitmq * Adds test cases for kafka * add test cases for rabbit sender * adds tests for received
Owner
|
@rajadilipkolli Thank you so much for the hard work. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.