You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2021/07/02 15:15:15 UTC
[GitHub] [trafficserver] bneradt opened a new pull request #8033: Treat Sphinx doc build warnings as errors.
bneradt opened a new pull request #8033:
URL: https://github.com/apache/trafficserver/pull/8033
It's easy to introduce warnings in the doc build without noticing them
due to the amount of output from the build. This changes the
sphinx-build invocation to treat any warnings we introduce as errors so
that such problems will get caught.
It also fixes a handful of warnings that got introduced recently as
well. With this patch, the build runs cleanly without warnings.
--
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: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] shinrich commented on pull request #8033: Treat Sphinx doc build warnings as errors.
Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #8033:
URL: https://github.com/apache/trafficserver/pull/8033#issuecomment-873076574
Looks good to me! Thanks for cleaning it up.
--
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: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] shinrich commented on a change in pull request #8033: Treat Sphinx doc build warnings as errors.
Posted by GitBox <gi...@apache.org>.
shinrich commented on a change in pull request #8033:
URL: https://github.com/apache/trafficserver/pull/8033#discussion_r663088137
##########
File path: doc/developer-guide/api/functions/TSSslSecret.en.rst
##########
@@ -67,7 +67,7 @@ Synopsis
#include <ts/ts.h>
-.. function:: TSReturnCode TSSslSecretGet(const char * secret_name, int secret_name_length)
+.. function:: TSReturnCode TSSslSecretUpdate(const char * secret_name, int secret_name_length)
Review comment:
Ah, good catch of a cut-n-paste error.
--
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: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bneradt merged pull request #8033: Treat Sphinx doc build warnings as errors.
Posted by GitBox <gi...@apache.org>.
bneradt merged pull request #8033:
URL: https://github.com/apache/trafficserver/pull/8033
--
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: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org