mumrah commented on code in PR #18183:
URL: https://github.com/apache/kafka/pull/18183#discussion_r1887112385


##########
.github/workflows/build.yml:
##########
@@ -118,6 +118,13 @@ jobs:
         # --no-scan:      For public fork PRs, we won't attempt to publish the 
scan
         run: |
           ./gradlew --build-cache --info $SCAN_ARG check -x test
+          ./gradlew siteDocTar

Review Comment:
   I think @chia7712's comment was to move the siteDocTar into this "Compile 
and validate" and add a subsequent step to verify the contents (e.g., "Check 
Site Docs"). Having two gradle commands in one step makes it harder to see what 
failed without digging into the logs.



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