Skip to content
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

Upgrade to Spring Boot 3.2.1; remove explicit logback dependency #130

Closed
wants to merge 1 commit into from

Conversation

rtitle
Copy link
Contributor

@rtitle rtitle commented Jan 4, 2024

Basically un-does #127, now that Spring Boot has upgraded to a patched logback-core.

Copy link

sonarqubecloud bot commented Jan 4, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@@ -14,7 +14,7 @@ plugins {
// removed this plugin and filled in specific version numbers. I also removed all
// version numbers with `+` in them. Was:
// id 'io.spring.dependency-management' version '1.0.10.RELEASE'
id 'org.springframework.boot' version '3.1.2'
id 'org.springframework.boot' version '3.2.1'
Copy link
Contributor Author

@rtitle rtitle Jan 4, 2024

Choose a reason for hiding this comment

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

Any downside in moving TCL to 3.2? I didn't see anything that would drastically affect us.

https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.2-Release-Notes

@rtitle rtitle requested review from dvoet and pshapiro4broad January 4, 2024 21:10
@rtitle
Copy link
Contributor Author

rtitle commented Jan 4, 2024

Hm tests failed, maybe there is some issue with the new dependencies. Will dig more later..

@cahrens
Copy link
Contributor

cahrens commented Feb 27, 2024

@rtitle, we are now at version 3.2.3 (my dependabot ignore was incorrect), so I'm trying this again in #156.

@cahrens cahrens closed this Feb 27, 2024
@pshapiro4broad pshapiro4broad deleted the rt-upgrade-spring-boot branch February 4, 2025 18:21
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