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
Remove JSR 305 annotations #2408
Conversation
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.
Let's use the jakarta.annotation-api
that provides Nullable
and Nonnull
rather than rolling our own, unless there's a reason not to.
replaced custom annotation with jakarta. |
The findbugs jsr305 annotations jar uses annotations in the javax.annotation namespace, which causes problems with JPMS. Remove the annotations from the code base. The Jdbi code itself transitions to error prone and spotbugs. For code that uses Jdbi (and all tests), the Jakarta annotation-api `jakarta.annotation.Nullable` is used as other candidates either do not have Runtime retention (spotbugs, jetbrains), have problems with JPMS (jsr305), or are very specialized (spring) or obscure (checker framework).
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
As discussed in #2408, we should prefer the `jakarta` annotations, but that's not what actually got committed here. So clean that up
As discussed in #2408, we should prefer the `jakarta` annotations, but that's not what actually got committed here. So clean that up
As discussed in #2408, we should prefer the `jakarta` annotations, but that's not what actually got committed here. So clean that up
As discussed in #2408, we should prefer the `jakarta` annotations, but that's not what actually got committed here. So clean that up
As discussed in #2408, we should prefer the `jakarta` annotations, but that's not what actually got committed here. So clean that up
As discussed in #2408, we should prefer the `jakarta` annotations, but that's not what actually got committed here. So clean that up
The findbugs jsr305 annotations jar uses annotations in the
javax.annotation namespace, which causes problems with JPMS.
Remove the annotations from the code base. The Jdbi code itself
transitions to error prone and spotbugs.
For code that uses Jdbi (and all tests), a new
annotation (org.jdbi.v3.core.mapper.reflect.Nullable) is introduced as
all candidates either do not have Runtime retention (spotbugs,
jetbrains), have problems with JPMS (jsr305), are very
specialized (spring) or obscure (checker framework).