minfrin commented on a change in pull request #5: Switch from XhtmlSink to Xhtml5Sink as the default site renderer. URL: https://github.com/apache/maven-doxia-sitetools/pull/5#discussion_r291078154
########## File path: doxia-site-renderer/pom.xml ########## @@ -161,13 +165,13 @@ under the License. <dependency> <groupId>net.sourceforge.htmlunit</groupId> <artifactId>htmlunit</artifactId> - <version>2.7</version> + <version>2.24</version> Review comment: Not practical alas, as explained in the commit the change in htmlunit required a change to the xhtml5 patch to account for changes in htmlunit behaviour. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services