kennknowles commented on code in PR #30635:
URL: https://github.com/apache/beam/pull/30635#discussion_r1526495962


##########
sdks/java/io/google-cloud-platform/build.gradle:
##########
@@ -116,8 +116,8 @@ dependencies {
   permitUnusedDeclared library.java.grpc_google_cloud_pubsublite_v1 // 
BEAM-11761
   implementation library.java.guava
   implementation library.java.http_client
-  provided library.java.hamcrest
-  permitUnusedDeclared library.java.hamcrest // BEAM-11761
+  implementation library.java.hamcrest
+  permitUnusedDeclared library.java.hamcrest // Needed for 
PipelineOptionRegistrar auto-service that registered TestPipelineOptions 
interfaces

Review Comment:
   This is interesting. Somehow I though it was a direct dependency within the 
code. Is it actually something upstream that has hamcrest as a provided 
dependency still?



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to