| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Extracted from CPython @ 1bced5656 Added a few more theme config values, split the donate footer into its own template, and removed CPython-docs sepcific JavaScript and HTML.
|
@brettcannon FYI I have a working version of the CPython docs that use this theme (and removes duplicated code), should I send that over to CPython after this is merged? :) |
Sorry, something went wrong.
There was a problem hiding this comment.
Mostly looks good to me, just the extra README paragraph requested, and a question about the use of the io module in setup.py
Sorry, something went wrong.
| Python Docs Sphinx Theme | ||
| ========================= | ||
|
|
||
| This is the theme for the Python documentation |
There was a problem hiding this comment.
The note from the python-dev discussion can just be added here as a second paragraph:
Note that when adopting this theme, you're also borrowing an element of the trust and credibility established by the CPython core developers over the years. That's fine, and you're welcome to do so for other Python community projects if you so choose, but please keep in mind that in doing so you're also choosing to accept some of the responsibility for maintaining that collective trust.
Sorry, something went wrong.
There was a problem hiding this comment.
Done, thank you. :)
Sorry, something went wrong.
| from setuptools import setup | ||
|
|
||
| # README into long description | ||
| with io.open('README.rst', encoding='utf-8') as readme_file: |
There was a problem hiding this comment.
Are we planning to support installing this theme for use in Py2 docs builds? If not, we can just use the open() builtin directly here, rather than going through the io module.
Sorry, something went wrong.
There was a problem hiding this comment.
Although this is technically for the Python 3 docs, using io.open over open is pretty cheap for us, especially considering this package contains almost no Python code. :)
Sorry, something went wrong.
|
@ncoghlan please take another look when you get a chance. :) |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks good to me!
Sorry, something went wrong.
|
Thanks!
…On Thu, Jun 8, 2017, 8:21 PM Nick Coghlan ***@***.***> wrote:
Merged #1 <#1>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAPUc0au4b50AlsiQuHwRBl9BqAy0Oa_ks5sCLozgaJpZM4NwmdZ>
.
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Extracted from CPython @ 1bced5656
Added a few more theme config values, split the donate footer into its own template, and removed CPython-docs sepcific JavaScript and HTML.