-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move Sphinx plugins and format to Python package #2379
Conversation
I don't fully understand this one. |
By separating |
That does sound like a good idea. I would suggest to just go for it. |
Someone would need to create a target repository in the JuliaLang account. I don't have the permission. |
I somehow missed this. Sorry about that. Should I create a |
Should this repository be a julia package itself? |
That's an interesting thought--I'm not quite sure how that would work, though. I'll need to look into how to use external resources on RTFD. |
Right, so on further reflection, that doesn't make any sense--it contains Python code, not Julia code, and it's used by Sphinx, not the package. However, it would make sense to make it installable by |
In
|
Sorry - that is a bit cryptic for me. Where do I create |
I created JuliaDoc, and you should have commit access. |
@ViralBShah thanks. The |
This was almost too easy: https://julia-docrepo.readthedocs.org/en/latest/ https://github.com/pao/julia/tree/doc-refactor Before this can be pushed, the owner of our RTFD instance needs to go in and turn the "Use virtualenv" option on, and set |
That would be me. @pao, if you give me your rtfd username, I can add you as an admin. |
Also, I made these changes. Might be easier if you're an admin though. |
Thanks, @StefanKarpinski; you'll be shocked to learn my RTFD username is "pao". I'll push this through tomorrow, test on one of my packages, and fill out the JuliaDoc README. Will also port #2123 over to the new repository. |
Oh, and a post to -dev so packagers who are using Sphinx for docs know this is available. |
Just realized our translators will need to do the same thing. cc @wlbksy @lbenitesanchez @andrioni you will need to update your RTFD instances per #2379 to keep the Julia document style for the latest manual. Your 0.1 documentation, if applicable, will continue to work as before. |
Oops, @wlbksy, see ^ (mistyped your username) |
received. would not find enough time until July. Thanks for noticing |
Between this and commits made to JuliaLang/JuliaDoc, this now supersedes #2123. |
For packages, looks like there's still a little work to do. Need to work out sidebar and proper hierarchy in topbar. https://monadsjl.readthedocs.org/en/latest/ is my guinea pig. |
Ignore the build failure. Something went wrong in the tests on clang but this doesn't touch any Julia code. GCC went fine. |
Styles didn't quite come through right from #2123. |
Style differences were due to insecure content loading restrictions in Firefox Nightly. I have pushed to fix that, but that will still leave problems with MathJax-rendered content if we have any, due to readthedocs/readthedocs.org#283. |
The links are specific to the main doc, so this really does belong here. Also move the translation links here.
Okay, I think I have this sorted out. There's still work to do to make this nice for packages, but this is serviceable and we can start working those issues on the JuliaDoc repo. @StefanKarpinski @ViralBShah @nolta if someone else can give this a once-over and it looks good feel free to merge it. |
...no one likes reviewing doc stuff, apparently. I'll merge this secure in the knowledge we can back it out. |
Move Sphinx plugins and format to Python package
I actually took a stab at implementing this documentation for the ODBC package and it was really easy! https://odbcjl.readthedocs.org/en/latest/ Looking forward to getting this widespread. |
@pao Sorry for not chiming in earlier - busy preparing a talk for a few days. Merging this was the right thing. |
Great! I intend to get a post up soon, but thanks for taking the plunge, @karbarcca! @ViralBShah, no problem, I know people are busy, |
BTW, |
Hmm. Unfortunately the Python packaging story is a messy business. Since we already require extra dependencies to build documentation, I'm inclined to just add pip (http:https://www.pip-installer.org/) to the list. |
Would it not work with |
Apparently it can, including installing directly from git and |
Mac has easy_install, but no pip. I guess this kind of thing would need to go into the Makefile. Have you announced this on the list and how people should use it and such? Perhaps the Example.jl package can be modified to use it. |
Haven't announced yet. Instructions are in the JuliaDoc README, and apparently they work if @karbarcca was able to get it going. |
I think the Juliadoc is pretty good, though honestly I just copied your
|
I'm trying to
Help plz? |
That output doesn't include anything about module installation--do you have I guess an alternative would be to pull it in as a submodule for the purposes of the main repository, and set |
@JeffBezanson and @ViralBShah, give f1427a6 a try? Hopefully there is no problem which cannot be solved with submodules and doesn't create more problems. |
Thanks. So much easier to use now! |
I just tried building the docs on master, and hit the issue of getting RTD installed first. I fixed it by doing a |
The change I suggested is merged (f1427a6), but only covers the Julia-specific extensions to Sphinx. You still need Sphinx. |
I see, my mistake. I have Sphinx installed, but on OS X, this still left me with a failed import for |
Ahh; I see that @nolta added that requirement recently with the new theme. |
Right now, if packages want to use ReadTheDocs but want to share style with mainline Julia, it's a copy-and-paste job. I did this to get the
julia
domain for the StrPack.jl docs, but it would be better just to add a submodule dependency so we can track changes made to the upstream documentation format and keep the style in sync.