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

com.google.code.findbugs:jsr305 can probably be removed #181

Open
cies opened this issue Jun 1, 2023 · 5 comments
Open

com.google.code.findbugs:jsr305 can probably be removed #181

cies opened this issue Jun 1, 2023 · 5 comments
Assignees

Comments

@cies
Copy link
Member

cies commented Jun 1, 2023

Since we use guava version 32.x.x.

Like here: playframework/playframework@d6cccba

@cies
Copy link
Member Author

cies commented Jun 5, 2023

Guava migrated to https://mvnrepository.com/artifact/org.checkerframework/checker-qual/3.35.0 (210k where jsr305 was 12k).

Internally we use org.jetbrains:annotations (29k).

All annotations that the Kotlin compiler understands are listed here:

https://github.com/JetBrains/kotlin/blob/master/core/compiler.common.jvm/src/org/jetbrains/kotlin/load/java/JvmAnnotationNames.kt

I can make a PR replacing the use of jsr305 with whatever is chosen.

@asolntsev
Copy link
Contributor

@cies Yes, why not. We can upgrade to whatever newer library. I don't see any reasons to prefer one of them over others.

But I personally like annotations with javax. prefix (like javax.annotation.Nullable) - just because they look like a standard. :)

@cies
Copy link
Member Author

cies commented Aug 14, 2023

I was mistaken. Guava has not migrated yet. The discussion here:

google/guava#2960

That thread is linked to by several other projects, and they usually migrate to jetbrains-annontations or checker-qual.

@cies
Copy link
Member Author

cies commented Aug 14, 2023

I'd opt for snoozing this until Guava makes the switch and then follow their choice (not to add any more dependencies).

@cies
Copy link
Member Author

cies commented Sep 17, 2024

Guava is still not decided. Lol.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants