Skip to content

Migrate to Spring 7 / Spring Boot 4#278

Merged
sonus21 merged 1 commit intosonus21:masterfrom
ITman1:boot-4
Feb 3, 2026
Merged

Migrate to Spring 7 / Spring Boot 4#278
sonus21 merged 1 commit intosonus21:masterfrom
ITman1:boot-4

Conversation

@ITman1
Copy link
Copy Markdown
Contributor

@ITman1 ITman1 commented Jan 22, 2026 •
edited
Loading

Description

Spring 7 and Boot 4 release which brings new major dependencies, mainly Jackson 3.

Type of change

Please delete options that are not relevant.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update?

How Has This Been Tested?

  • Unit Test
  • Integration Test
  • Reactive Integration Test
  • I built SNAPSHOT and tested it in our codebase. Our integration test with RqueueMessageEnqueuer passed.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes

@ITman1 ITman1 mentioned this pull request Jan 22, 2026
Open
@ITman1
Copy link
Copy Markdown
Contributor Author

ITman1 commented Jan 22, 2026

Tests passed locally, only 2 failed, but they are marked as local. There are 4 reported as failed because there si re-try for them.

image

@ITman1
Copy link
Copy Markdown
Contributor Author

ITman1 commented Jan 22, 2026 •
edited
Loading

Another test run:
image

MessageProcessorTest failed once, but succeeded on re-try.

@sonus21
Copy link
Copy Markdown
Owner

sonus21 commented Jan 22, 2026

Thanks for sharing the pr. I'll review and update the PR, let me see why CI is not running.

@ITman1
Copy link
Copy Markdown
Contributor Author

ITman1 commented Jan 22, 2026

Thanks for sharing the pr. I'll review and update the PR, let me see why CI is not running.

Thank you. FYI I removed Spring dependency management ("for now?"), it was giving me some exception that configuration was already applied, did not investigate further more.

@ITman1
Copy link
Copy Markdown
Contributor Author

ITman1 commented Jan 29, 2026

@sonus21 Any finding why CI is not running?

@sonus21
Copy link
Copy Markdown
Owner

sonus21 commented Jan 29, 2026

@ITman1 can you take a pull again and push again please? I'm not sure why this has issue.

@ITman1 ITman1 force-pushed the boot-4 branch 2 times, most recently from 16fd2ea to 886e22d Compare January 29, 2026 20:14
@ITman1
Copy link
Copy Markdown
Contributor Author

ITman1 commented Jan 29, 2026

@sonus21 Push helped, I did also minor fixes in reactive tests. CI is failing on last step - coveralls, no clue how to fix it quickly, seems like some unmaintained plugin, last release 5 years ago.

@sonus21
Copy link
Copy Markdown
Owner

sonus21 commented Jan 30, 2026

Thanks for checking, let me check what's going on. @ITman1

@sonus21
Copy link
Copy Markdown
Owner

sonus21 commented Feb 3, 2026

I'm merging this pr, I'll take a pull from main and check what's going on.

@sonus21 sonus21 merged commit 427bae5 into sonus21:master Feb 3, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ITman1 @sonus21