andygrove commented on code in PR #2476: URL: https://github.com/apache/datafusion-ballista/pull/2476#discussion_r4076242373
########## docs/build.sh: ########## @@ -18,4 +18,4 @@ # under the License. rm -rf build -make html +SPHINXOPTS="-W --keep-going" make html Review Comment: I don't think we should fail the docs build on warnings unless we have a PR check to ensure that we don't merge PRs that introduce warnings. Our docs would just stop getting published in CI -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
