[GitHub] [calcite] vlsi commented on a change in pull request #2641: [CALCITE-4934] Deploy snapshots releases on master merge

[GitBox <gi...@apache.org>]

vlsi commented on a change in pull request #2641:
URL: https://github.com/apache/calcite/pull/2641#discussion_r767423885



##########
File path: build.gradle.kts
##########
@@ -806,6 +806,18 @@ allprojects {
                 // Do not publish "root" project. Java plugin is applied here for DSL purposes only
                 return@configure
             }
+
+            repositories {
+                maven {
+                    name = "ApacheSnapshot" //  optional target repository name
+                    url = uri("https://repository.apache.org/content/repositories/snapshots")
+                    credentials {
+                        username = (project.properties["nexusSnapshotUser"] ?: "").toString()
+                        password = (project.properties["nexusSnapshotPassword"] ?: "").toString()
+                    }
+                }
+            }

Review comment:
       The repository is already configured, so there's no need to add one more




-- 
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: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org