[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added inline comments. Comment at: docs/analyzer/checkers.rst:1946 +Checkers used for debugging the analyzer. +:doc:`DebugChecks` page contains a detailed description. + NoQ wrote: > The error is on this line. It might be that this link should mention t

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ added a comment. Dunno but made a random guess in the inline comment. Comment at: docs/analyzer/checkers.rst:1946 +Checkers used for debugging the analyzer. +:doc:`DebugChecks` page contains a detailed description. + The error is on this line. It might be t

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added a comment. I'm getting a single buildbot failure, but I'm a little unsure how to fix it. Could you help? http://lab.llvm.org:8011/builders/clang-sphinx-docs/builds/36985 24.610 [1/1/1] Generating html Sphinx documentation for clang into "/home/buildbot/llvm-build-dir/clang-sp

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added a comment. I'll do the honors. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54429/new/ https://reviews.llvm.org/D54429 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/l

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ added a comment. P.S. I also support the idea to have a separate page per checker. It encourages people to write as much documentation as necessary and continously expand it. Otherwise there's a perception that all docs need to be roughly of the same size and shape in order to make the page

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. I think let's get this up and running and slowly redirect links from the front page to the new docs until only the front page is left? Comment at: docs/analyzer/checkers.rst:36-37

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added a comment. @NoQ @dcoughlin @george.karpenkov Any objections to this? :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54429/new/ https://reviews.llvm.org/D54429 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:/

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added a comment. Cheers, this is as good as it gets! IRL we also mentioned making a unipage for each checker which would be awesome (and would require a lot of tedious work), but as a start, I'd much prefer browsing through this doc than the current one. Very much appreciated! I won'

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added a comment. That I think sounds great! Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54429/new/ https://reviews.llvm.org/D54429 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mai

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ added a comment. I poked Devin offline. It kinda seems to both of us that Sphinx is great for documentation that is updated regularly, but our fancy front page should probably stay. Similarly to how we still have the llvm.org front page, despite all the documentation. It's kinda atmospheric

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

dkrupp added a comment. @dcoughlin @NoQ ping... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54429/new/ https://reviews.llvm.org/D54429 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

dkrupp added a comment. Herald added a subscriber: gamesh411. @dcoughlin could you please look into this? https://reviews.llvm.org/D54429 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

dkrupp updated this revision to Diff 173846. dkrupp added a comment. -scanbuild and xcode pictures are included now -intro text ("What is Static Analysis?" etc.) are put under the Introduction section -Download section is created, but I am not sure how well was the this Mac OSX binary release se

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ added a reviewer: dcoughlin. NoQ added a subscriber: dcoughlin. NoQ added a comment. I think we should escalate this to cfe-dev first; this sounds pretty important. @dcoughlin should definitely say a word here, as the code owner. I agree that pictures and screenshots are very important to ke

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

Szelethus added a comment. In https://reviews.llvm.org/D54429#1295838, @george.karpenkov wrote: > What do you propose we should do with other pages on the existing website? > (OpenProjects / etc.) I think we should just move everything here, and forget about the old site, as there clearly isn

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

george.karpenkov added a comment. From a first glance, it's missing screenshots and an introduction section. What do you propose we should do with other pages on the existing website? (OpenProjects / etc.) Repository: rC Clang https://reviews.llvm.org/D54429 _