Claudenw commented on code in PR #333: URL: https://github.com/apache/creadur-rat/pull/333#discussion_r1779464840
########## .buildtools/generateStagingSiteInWebpageRepo: ########## @@ -14,6 +14,6 @@ # limitations under the License. # # DEVHINT: RAT-302: Do not use modern JDK due to the javadoc problem; JDK8 and 16 are okay -./mvnw clean site:site site:stage +./mvnw clean package site site:stage -DskipTests Review Comment: I did some testing this morning. - We need `package` because without it the final jar in the `apache-rat` module is not generated, and that jar is used in the `site:pre-site` processes to generate the included files for the `*.apt.vm` processing. - We need `site` because that generates the site for all the `apache-rat-*` modules. Without that the `site:stage` fails. - We need `site:stage` to get a complete staged site. I will assume you are correct and remove the `-D skipTests` -- 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]
