Skip to content

Commit

Permalink
Exclude error prone annotations from the build
Browse files Browse the repository at this point in the history
The annotations are not required by Guice at runtime, see
google/guice#1739
  • Loading branch information
ThePumpingLemma committed Jul 6, 2023
1 parent a81b4e6 commit f2f53e3
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 4 deletions.
4 changes: 4 additions & 0 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,10 @@ subprojects {
resolutionStrategy.capabilitiesResolution.withCapability("com.google.guava:listenablefuture") {
select("com.google.guava:guava:0")
}

// Omit optional dependency from Guice + Guava.
// https://github.com/google/guice/pull/1739
exclude(group = "com.google.errorprone", module = "error_prone_annotations")
}
}

Expand Down
1 change: 0 additions & 1 deletion buildSrc/src/main/kotlin/Dependencies.kt
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ object Dependencies {
val dockerTransport = "com.github.docker-java:docker-java-transport:3.3.0"
val dockerTransportHttpClient = "com.github.docker-java:docker-java-transport-httpclient5:3.3.0"
val dokkaGradlePlugin = "org.jetbrains.dokka:dokka-gradle-plugin:1.6.10"
val errorproneAnnotations = "com.google.errorprone:error_prone_annotations:2.20.0"
val findBugs = "com.google.code.findbugs:jsr305:3.0.2"
val gax = "com.google.api:gax:2.5.0"
val gcpCloudCore = "com.google.cloud:google-cloud-core:1.95.4"
Expand Down
1 change: 0 additions & 1 deletion misk-aws-dynamodb-testing/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ dependencies {
api(Dependencies.tempestTestingInternal)
api(project(":misk-aws-dynamodb"))
api(project(":misk-inject"))
implementation(Dependencies.errorproneAnnotations)
implementation(Dependencies.kotlinReflect)
implementation(Dependencies.tempestTesting)
implementation(Dependencies.tempestTestingDocker)
Expand Down
1 change: 0 additions & 1 deletion misk-aws2-dynamodb-testing/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ dependencies {
api(project(":misk-aws2-dynamodb"))
api(project(":misk-inject"))
api(project(":misk-testing"))
implementation(Dependencies.errorproneAnnotations)
implementation(Dependencies.tempest2Testing)
implementation(Dependencies.tempest2TestingDocker)
implementation(Dependencies.tempest2TestingJvm)
Expand Down
1 change: 0 additions & 1 deletion misk-clustering/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ dependencies {
api(Dependencies.wispConfig)
api(Dependencies.wispLeaseTesting)
api(project(":misk-inject"))
implementation(Dependencies.errorproneAnnotations)
implementation(Dependencies.guice)
implementation(Dependencies.kubernetesClient)
implementation(Dependencies.kubernetesClientApi)
Expand Down

0 comments on commit f2f53e3

Please sign in to comment.