Re: Run Sphinx checks on MXNet CI

2018-11-11 Thread Anirudh Acharya
Thanks for the reply Aaron. Once the existing Sphinx errors are fixed and codebase is cleaned up, lets definitely revisit this and try to add a Sphinx build into the CI pipeline so that we can prevent MXNet documentation from breaking. Thanks Anirudh On Thu, Nov 8, 2018 at 5:16 PM Aaron Markham

Re: Run Sphinx checks on MXNet CI

2018-11-08 Thread Aaron Markham
Hi Anirudh, Once the existing errors in docs building are cleaned up, I'm all for having CI bubble up a build break when docs are broken by a PR. That way we're keeping things up to date and not letting a minor bug turn into a serious issue for the entire API documentation. One break causes a

Run Sphinx checks on MXNet CI

2018-11-08 Thread Anirudh Acharya
Hi, Recently there was a barrage of issues related to documentation that was raised here - https://github.com/apache/incubator-mxnet/issues/created_by/aaronmarkham All the issues are related to Sphinx errors and warnings. These errors often lead to broken documentation. Ideally such errors should