Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

Adding Sphinx for Documentation #36

Open
wants to merge 59 commits into
base: main
Choose a base branch
from

Conversation

Lementknight
Copy link
Contributor

I am currently working on integrating Sphinx, so that we can facilitate better documentation building.

docs/index.rst Outdated Show resolved Hide resolved
docs/conf.py Outdated
Comment on lines 41 to 43
'jupyter_sphinx',
'reno.sphinxext',
'sphinx.ext.intersphinx',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be deleted

docs/conf.py Outdated
Comment on lines 79 to 82
# Intersphinx configuration
intersphinx_mapping = {
'qiskit': ('https://qiskit.org/documentation/', None),
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be deleted too

@mtreinish
Copy link
Member

Once you apply this change I think this is ready to go for a default config. There isn't any content yet but we can add that later (along with a ci config to build documentation on each commit).

diff --git a/tox.ini b/tox.ini
index 95ed6e7..3e3e8b7 100644
--- a/tox.ini
+++ b/tox.ini
@@ -34,5 +34,8 @@ deps =
 commands = black {posargs} red_queen/
 
 [testenv:docs]
+deps =
+    qiskit_sphinx_theme
+    -r{toxinidir}/requirements.txt
 commands =
   sphinx-build -b html {posargs} docs/ docs/_build/html

@Lementknight
Copy link
Contributor Author

Where do I make this change?

Lementknight and others added 5 commits August 12, 2022 15:00
In this commit I added the sphinx package to the requirements.txt file in the red-queen directory
I added dependencies to the tox.ini in the [testenv:docs] section
In this commit I removed spinx from this requirements.txt file as it is now included due to the previous commit
I added Sphinx to the list of dependencies in for [testenv:docs] in the tox.ini folder
@Lementknight Lementknight marked this pull request as ready for review August 12, 2022 19:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants