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


##########
.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:
   That's exactly what I meant - instead of "mvn site site:stage" shouldn't 
"mvn site:stage" be enough?
   
   Is the stage build correct as the pom does not contain a valid site URL:
     <distributionManagement>
       <site>
         <id>staging</id>
         <url>file:bogus</url>
       </site>
     </distributionManagement>
   
   Therefore it was not used before.



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