ottlinger commented on code in PR #184:
URL: 
https://github.com/apache/maven-doxia-sitetools/pull/184#discussion_r1953159702


##########
doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java:
##########
@@ -700,8 +700,9 @@ public SiteRenderingContext createContextForSkin(
                 if (StringUtils.isNotBlank(toolsPrerequisite)
                         && (current != null)
                         && !matchVersion(current, toolsPrerequisite)) {
-                    throw new RendererException("Cannot use skin: has " + 
toolsPrerequisite
-                            + " Doxia Sitetools prerequisite, but current is " 
+ current);
+                    throw new RendererException("Cannot use skin as it needs " 
+ toolsPrerequisite
+                            + " Doxia Sitetools, but your configuration uses " 
+ current + "."
+                            + "Check your version setup of plugins and skins 
to use the same doxia stack.");

Review Comment:
   I ran into this error as my combination of site-plugin-version and skin 
version did not match. It's not only a matter of the skin, but I will open a 
new PR with the changes you proposed and we'll see if that gets merged. Thanks.



-- 
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

Reply via email to