Claudenw commented on code in PR #333:
URL: https://github.com/apache/creadur-rat/pull/333#discussion_r1778101006


##########
.github/workflows/maven.yml:
##########
@@ -68,4 +68,4 @@ jobs:
         run: ./mvnw -e -B -V -ntp javadoc:javadoc
 
       - name: Build site
-        run: ./mvnw -e -B -V -ntp site
+        run: ./mvnw -e -B -V -ntp site site:stage

Review Comment:
   From the site plugin documentation:
   
   
```[site:stage](https://maven.apache.org/plugins/maven-site-plugin/stage-mojo.html)
 generates a site in a local staging or mock directory based on the site URL 
specified in the <distributionManagement> section of the POM. It can be used to 
test that links between module sites in a multi module build work. This goal 
requires the site to already have been generated using the site goal, such as 
by calling mvn site.```
   
   So I think `site site:stage` is correct. 



-- 
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: dev-unsubscr...@creadur.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to