olamy commented on a change in pull request #51:
URL: https://github.com/apache/maven-resolver/pull/51#discussion_r427743790
##########
File path: maven-resolver-transport-wagon/pom.xml
##########
@@ -56,7 +56,7 @@
<dependency>
<groupId>org.apache.maven.wagon</groupId>
<artifactId>wagon-provider-api</artifactId>
- <version>3.4.0</version>
+ <version>3.4.1-SNAPSHOT</version>
Review comment:
I don't understand the problem??
This sounds complicated to not depends on a SNAPSHOT?? at least until we did
enough testing and wagon get released.
I'm totally fine as it will be continuous testing as master of our projects
are deployed as SNAPSHOT for every commits.
How do you expect to build/test everything together in CI system???
Or even how do you except reviewing this PR??
we will just mark this as dependant of wagon SNAPSHOT if we accept both for
sure.
At least it makes it easy to test it locally because here we have an issue
which need to build 3 projects before being able to test everything. So yes
using SNAPSHOT will make it easier
----------------------------------------------------------------
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]
