kwin commented on code in PR #630:
URL: 
https://github.com/apache/maven-doxia-sitetools/pull/630#discussion_r3008600523


##########
doxia-integration-tools/src/main/java/org/apache/maven/doxia/tools/SiteTool.java:
##########
@@ -129,6 +134,32 @@ SiteModel getSiteModel(
             List<RemoteRepository> remoteProjectRepositories)
             throws SiteToolException;
 
+    /**
+     * Get a site model for a project.
+     *
+     * @param siteDirectory the site directory, may be null if project from 
repository
+     * @param locale the locale used for the i18n in SiteModel, not null.
+     * See {@link #getSiteDescriptor(File, Locale)} for details.
+     * @param project the Maven project, not null.
+     * @param request the Maven execution request (may be null). This is 
needed to get the user properties and the system properties for interpolation.

Review Comment:
   Fixed in 
https://github.com/apache/maven-doxia-sitetools/pull/630/commits/ce678e3e8fb943ad78c3f24b24366ef34a90dbb4.



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

Reply via email to