feat!: support of Spring Boot 4 - #65
Conversation
4e0c7e8 to
6ad9781
Compare
|
@tgrafford Thank you for the PR! I haven't looked at this repository in a long while, and only have done occasional maintenance on it since the original creator stepped away. I don't work actively in Java these days, so I strongly appreciate that you took the time to do a version upgrade like this. This repo needs a few Github Actions upgrades and bugfixes to go in before this PR will go green on CI, which I'm working on presently. Once those are in, I'll review the PR, and we should be able to go from there. Apologies for not noticing the PR sooner-- my Github notifications are flooded at all times, so this PR slipped through the cracks. |
philipaconrad
left a comment
There was a problem hiding this comment.
I'm not particularly familiar with Spring Boot bits, but the OPA-related side of the changes seem fine, and the nullable annotations look good as well. I'm marking approve, so that we can finally get this set of changes rolled in. Thanks again @tgrafford!
BREAKING CHANGE: library no longer works with Spring Boot 3 Signed-off-by: Tim Grafford <tim@grafford.dev>
6ad9781 to
cb07d67
Compare
|
ℹ️ Rebasing the PR through the Github Web UI to catch up with |
BREAKING CHANGE: library no longer works with Spring Boot 3
This PR closes issue #59.
🔩 What code changed, and why?
Spring Boot upgraded to version 4.0.3 (latest as of writing). This included:
👍 Definition of done
👟 How to test
The testing assumes you already have a Spring Boot application using opa-springboot and the application has been upgraded to Spring Boot 4 but does not compile.
gradle.propertiestoversion=2.0.0-LOCAL./gradlew clean build publishToMavenLocal2.0.0-LOCALNote: Ensure your application resolves dependencies from your local maven registry.
./gradlew bootRunor./mvn spring-boot:rundepending on build tool used.⛓️ Related Resources