-
-
Notifications
You must be signed in to change notification settings - Fork 65
Make this available in PyPI? #5
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
Comments
I suggested this for both this repo and its downstream pypa-docs-theme, but @ncoghlan raised some concerns. |
There's https://pypi.python.org/pypi/python-docs-theme/ but with no files, do you remember the conerns? |
IIRC, my main question was just asking what we'd gain from the extra update publication overhead relative to having users install from the git repo. I'm OK with "We want to set a good example" as an answer to that question, though :) |
There is another (IMHO blocking) reason to make this available on PyPI: as it stands, if the changes associated with bpo-30607 / cPython PR 2017 were merged, this would introduce a new dependency on Git in the cPython Docs build. Currently, there is no such dependency and, in fact, I know of at least one build process that depends on there being no dependency on Git. This would also affect any builders from cPython source tarballs, unless the release process were changed to bundle the docs theme into the source tarball. I would much prefer to see this as a PyPI download like the rest of the Docs build toolchain (e.g. Sphinx and friends). |
Thanks for asking :) Sure, feel free to add me. |
Aye, happy to be a backup publisher :) |
Great. It occurs to me that we need to think of a versioning strategy. Should we just match Python major.minor? |
I'd suggest a date based version - that way if we decide to update the look & feel, we can make the decision on whether or not to update old maintenance branches at the same time. |
SGTM. Created #11 and closing this as we've published. :) |
No description provided.
The text was updated successfully, but these errors were encountered: