Skip to content
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

Docs in Sphinx #1526

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

Docs in Sphinx #1526

wants to merge 17 commits into from

Conversation

rpkak
Copy link
Contributor

@rpkak rpkak commented Jul 29, 2021

I moved the documentation from jekyll to sphinx (@nvie #111).

This PR is not finished yet (you see that this is a draft PR), because I will change some things in the content. So @selwin you can review the layout.

For this PR I created the folder docssphinx. In the end I will move this to docs.

@selwin because there are other PRs that change the doc, I think the best option is, that I implement the new docs after you approved this PR.

Closes #111

temporary to docssphinx

docs/index.md is divided up to docssphinx/index.rst and
docssphinx/gettingstarted.rst

portation was done by pandoc in most times
with darkmode
@codecov
Copy link

codecov bot commented Jul 29, 2021

Codecov Report

Merging #1526 (3118a14) into master (77e7ef6) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 3118a14 differs from pull request most recent head 382f8c6. Consider uploading reports for the commit 382f8c6 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1526   +/-   ##
=======================================
  Coverage   95.21%   95.22%           
=======================================
  Files          46       46           
  Lines        6483     6494   +11     
=======================================
+ Hits         6173     6184   +11     
  Misses        310      310           
Impacted Files Coverage Δ
rq/job.py 98.25% <ø> (ø)
rq/worker.py 88.59% <ø> (ø)
tests/test_queue.py 100.00% <0.00%> (ø)
tests/test_registry.py 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77e7ef6...382f8c6. Read the comment docs.

@selwin
Copy link
Collaborator

selwin commented Jul 31, 2021

Hey, so the issue you're referring too is super old. Back then, the docs was still hosted on gh-pages branch but I eventually merged it into the master branch.

I much prefer markdown than RST and I don't have any complaints about the docs site being built in Jekyll.

@rpkak
Copy link
Contributor Author

rpkak commented Jul 31, 2021

The biggest thing why I made this PR was because readthedocs has version control and so we wouldn't always have to add New in version 1.9.0. like here.

You also don't have to use the browser but you can generate pdf, epub or man pages as well.

Also look at the API Reference. This is also impossible for Jekyll.

If you prefer md I can port the content pages back to md but I wouldn't port files like docs/index.rst or api.rst, because they only contain sphinx contents like toctree or autoclass. I don't think that will be a problem because you can use markdown and restructuredtext at the same documentation.

Edit: I ported every restructuredtext file to MyST (Markedly Structured Text, Markdown but with sphinx options).
Edit2: Apart from api.rst

@rpkak
Copy link
Contributor Author

rpkak commented Aug 2, 2021

@selwin If you want to look at this documentation, I created a temporary readthedocs link here.

@rpkak rpkak marked this pull request as ready for review August 2, 2021 14:19
@selwin
Copy link
Collaborator

selwin commented Aug 3, 2021

Also look at the API Reference. This is also impossible for Jekyll.

I think this is the only real win for me.

I ported every restructuredtext file to MyST (Markedly Structured Text, Markdown but with sphinx options).

I think I still prefer using .md or some of the more well supported markdown flavors for portability purposes (maybe CommonMark or Github Flavored Markdown?). This will make it easier in case that we need to switch to a different docs infrastructure.

I looked at the test site you made; some of the styling is a bit messy so I suppose if we end up switching to Sphinx, we should use one of their more established themes.

My worry with this accepting a pull PR like this is that I don't fully understand Sphinx and the build process so it may be a bit troublesome to maintain in the long run.

Let me think about this a bit more. I will get around to reviewing your CLI PR sometime this weekend, I promise.

Thanks for all your efforts, tackling big issues like this. It's much appreciated.

@rpkak
Copy link
Contributor Author

rpkak commented Aug 3, 2021

@selwin

I think I still prefer using .md or some of the more well supported markdown flavors for portability purposes (maybe CommonMark or Github Flavored Markdown?). This will make it easier in case that we need to switch to a different docs infrastructure.

If you look at 0d3f7d8, you see that I switched to MyST. This is a Markdown flavor which supports rst features:

# Documentation
```{toctree}
:caption: 'Contents:'
:maxdepth: 2
queues
workers
results
jobs
exceptions
scheduling
job_registries
monitoring
connections
testing
```

I looked at the test site you made; some of the styling is a bit messy so I suppose if we end up switching to Sphinx, we should use one of their more established themes.

Ok, which themes do you prefer.

@selwin
Copy link
Collaborator

selwin commented Aug 7, 2021

If you look at 0d3f7d8, you see that I switched to MyST. This is a Markdown flavor which supports rst features:

I'm aware, but what I'm saying is I'd rather switch to common mark or Github flavored one for portability purposes (easier to move to a different docs infrastructure :).

Ok, which themes do you prefer.

I think Furo: https://sphinx-themes.org/sample-sites/furo/

@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
@rpkak rpkak marked this pull request as draft August 11, 2021 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Converting the existing gh-pages docs to Sphinx
2 participants