Bill commented on a change in pull request #5967:
URL: https://github.com/apache/geode/pull/5967#discussion_r566366660
##########
File path: geode-common/build.gradle
##########
@@ -26,6 +26,7 @@ dependencies {
implementation(platform(project(':boms:geode-all-bom')))
implementation('com.fasterxml.jackson.core:jackson-databind')
+ api('org.jetbrains:annotations')
Review comment:
I want to introduce a compile-time dependency on `annotation`, in any
subproject that depends on `geode-common`, because I want to save my colleagues
from having to add an explicit dependency on `annotation` to the `build.gradle`
for each subproject, as they desire to add a `@NotNull` annotation in some new
place.
I tried changing this to `compileOnly('org.jetbrains:annotations')` but now
the compiler/IDE says `java: package org.jetbrains.annotations does not exist`
where I'm trying to import it over in `SocketMessageWriter.java` in
`geode-core`.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
