rhoughton-pivot commented on a change in pull request #5967:
URL: https://github.com/apache/geode/pull/5967#discussion_r566375472
##########
File path:
buildSrc/src/main/groovy/org/apache/geode/gradle/plugins/DependencyConstraints.groovy
##########
@@ -272,5 +272,10 @@ class DependencyConstraints implements Plugin<Project> {
dependencySet(group: 'org.springframework.session', version: '2.4.1') {
entry('spring-session-data-redis')
}
+
Review comment:
Are we expecting to have more modules from `org.jetbrains` in the
product? If not, then the syntax above for individual modules is a little
cleaner.
##########
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:
@kohlmu-pivotal wanted the same thing: to have a java project or .gradle
file, whose only goal was to make it easy for other Geode subprojects to get
dependencies. Maybe he found a solution?
----------------------------------------------------------------
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]
