Copilot commented on code in PR #521:
URL:
https://github.com/apache/maven-build-cache-extension/pull/521#discussion_r3740819044
##########
pom.xml:
##########
@@ -217,16 +217,13 @@ under the License.
<version>${slf4jVersion}</version>
</dependency>
+ <!-- Not compiled against: ships the "dav" Wagon provider so that a remote
cache configured with a
+ dav: URL works. No Maven distribution bundles it, only
wagon-file/wagon-http. -->
Review Comment:
The new POM comment is meant to justify keeping the WebDAV wagon, but the
clause “only wagon-file/wagon-http” is overly specific and conflicts with the
PR description (which notes Maven distributions also carry other wagon
artifacts). To avoid future confusion if Maven’s bundled wagon set changes,
reword this to state the key point (WebDAV wagon isn’t bundled, and this
dependency provides the dav: provider at runtime) without enumerating bundled
wagons.
--
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]
