Skip to content

Conversation

@AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Sep 26, 2024

Opening for comment. Having the download page be more normal is probably for the best, but it does require a Sphinx extension, rather than "just" Jinja templating.

A


📚 Documentation preview 📚: https://cpython-previews--124614.org.readthedocs.build/

@AA-Turner AA-Turner added docs Documentation in the Doc dir skip issue skip news needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Sep 26, 2024
Comment on lines 72 to +77
rst_epilog = f"""
.. |archives_version| replace:: {version if 'daily' in tags else release}
.. |python_version_literal| replace:: ``Python {version}``
.. |python_x_dot_y_literal| replace:: ``python{version}``
.. |usr_local_bin_python_x_dot_y_literal| replace:: ``/usr/local/bin/python{version}``
"""
""" # noqa: F821
Copy link
Member Author

@AA-Turner AA-Turner Sep 26, 2024

Choose a reason for hiding this comment

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

@hugovk
Copy link
Member

hugovk commented Sep 27, 2024

I'm not too keen on this, it's replacing an HTML template with RST but at the cost of introducing a new Sphinx extension to maintain.

And the HTML isn't too complex.

@AA-Turner AA-Turner closed this Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core review docs Documentation in the Doc dir needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes skip issue skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants