Hello,
I was wondering if it would be possible to add <scope>test</scope> to the maven-model and maven-artifact dependencies introduced in commit eeae78a.
It looks as though both dependencies were intended to be test-scoped — they are only used in ExternalTest.java and are placed between other test-scoped Jetty dependencies in the POM. However, as they are currently declared without an explicit scope, they default to compile and leak transitively into downstream projects (together with plexus-utils as a transitive dep of maven-model).
A patch release addressing this would be much appreciated, as consumers of htmlunit 5.0.0 currently have to exclude these three artifacts manually to avoid polluting their compile classpath with Maven internals.
Thank you very much for your great work on this project!
Hello,
I was wondering if it would be possible to add
<scope>test</scope>to themaven-modelandmaven-artifactdependencies introduced in commit eeae78a.It looks as though both dependencies were intended to be test-scoped — they are only used in
ExternalTest.javaand are placed between othertest-scoped Jetty dependencies in the POM. However, as they are currently declared without an explicit scope, they default tocompileand leak transitively into downstream projects (together withplexus-utilsas a transitive dep ofmaven-model).A patch release addressing this would be much appreciated, as consumers of
htmlunit5.0.0 currently have to exclude these three artifacts manually to avoid polluting their compile classpath with Maven internals.Thank you very much for your great work on this project!