sman-81 commented on code in PR #60:
URL: https://github.com/apache/maven-jxr/pull/60#discussion_r930668016
##########
maven-jxr-plugin/src/it/JXR-143_nofork/pom.xml:
##########
@@ -44,6 +44,7 @@ under the License.
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
+ <version>3.1.0</version>
Review Comment:
> Current build on CI on master pass as I see ...
>
> rule `Always Pass` probably was added in order to override rules from
parent ... I will check it.
Hello @slawekjaranowski
I looked further into this. While running the integration tests I was behind
a proxy with access only to an internal Nexus artifact manager. As the plugin
has not version specified, the list of versions is fetched from Central - which
failed.
Therefore, strictly speaking the version is not required on this plugin for
the test to be green, but explicit versions are good practice and strongly
encouraged since at least Maven 3.
Maybe removing `maven-enforcer-plugin` from this test altogether is the best
resolution?
--
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: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
