slachiewicz commented on PR #681: URL: https://github.com/apache/maven-doxia-sitetools/pull/681#issuecomment-5326285066
Closing this one deliberately rather than letting it sit red. `resolverVersion` is paired with `mavenVersion`, not independent of it. Maven 3.6.3 declares resolver **1.4.1** in its own pom and Maven 3.9.9 declares **1.9.22**, so `mavenVersion=3.6.3` with `resolverVersion=1.4.1` is exactly the resolver the declared minimum Maven supplies. `maven-resolver-api` is `provided` scope, so building against 1.9.27 would let API absent from that baseline compile cleanly and fail at runtime instead. The eleven `SiteToolTest` errors are the visible half of the same thing: resolver 1.9.x restructured `DefaultRepositorySystem` to constructor injection, so Sisu can no longer satisfy `org.eclipse.aether.RepositorySystem` from the components on the test classpath. Moving to resolver 1.9.x is worth doing, but as a decision about the supported Maven baseline, raising `mavenVersion` alongside it. #682 adds an ignore rule so this stops arriving as routine dependency hygiene. *This comment was created with AI assistance.* -- 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]
