Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Sphinx configuration: remove unused sections #22

Merged
merged 1 commit into from
Jun 27, 2016

Conversation

svenkreiss
Copy link
Contributor

@svenkreiss svenkreiss commented Jun 24, 2016

Most importantly: it works around a problem with the viewcode extension on readthedocs.


This change is Reviewable

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.958% when pulling 1e964fb on simplify-sphinx-conf into 005fa36 on master.

@danvk
Copy link
Contributor

danvk commented Jun 27, 2016

I don't think I have enough context to totally understand this, but :lgtm:


Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.


docs/sphinx/conf.py, line 46 [r1] (raw file):

default_role = 'py:obj'

autodoc_member_order = "bysource"

single quotes


Comments from Reviewable

@svenkreiss
Copy link
Contributor Author

Review status: 2 of 3 files reviewed at latest revision, 1 unresolved discussion.


docs/sphinx/conf.py, line 46 [r1] (raw file):

Previously, danvk (Dan Vanderkam) wrote…

single quotes

Done.

Comments from Reviewable

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.958% when pulling 81d2c26 on simplify-sphinx-conf into 005fa36 on master.

@svenkreiss svenkreiss merged commit c9ac6e5 into master Jun 27, 2016
@svenkreiss svenkreiss deleted the simplify-sphinx-conf branch June 27, 2016 20:20
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.

None yet

3 participants