ctubbsii commented on code in PR #309:
URL: https://github.com/apache/accumulo-testing/pull/309#discussion_r3502677306
##########
pom.xml:
##########
@@ -121,19 +120,32 @@
<scope>test</scope>
</dependency>
</dependencies>
+ <repositories>
+ <repository>
+ <releases>
+ <enabled>false</enabled>
+ </releases>
+ <snapshots>
+ <enabled>true</enabled>
+ </snapshots>
+ <id>apache.snapshots</id>
+ <name>Apache Snapshot Repository</name>
+ <url>https://repository.apache.org/content/repositories/snapshots</url>
+ </repository>
+ </repositories>
Review Comment:
However, it is very risky to leave it in place. Having it enabled by default
across the board can get a user blocked by ASF. Enabling it requires some
understanding of the risks, and should be a conscious choice. Users can set it
in a maven profile or in their `MAVEN_OPTS` if they understand the risks and
wish to enable it. Or, they can choose to take a safer option and build a local
snapshot with `mvn install`, which should really be the default path, since
it's the safest, and most useful (it tests the current snapshot, not whichever
the last one that was published to the snapshot repo, which could be out of
date).
--
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]
