-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
[3.6] Doc: Add an optional obsolete header. (GH-13638). #13657
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
Conversation
(cherry picked from commit 46ed90d) Co-authored-by: Julien Palard <[email protected]>
@ned-deily Hi Ned, would you please merge this, if you like it? See screenshot here: #13638 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you also planning to add outdated
to conf.py
?
Doc/tools/templates/layout.html
Outdated
<div id="outdated-warning" style="padding: .5em; text-align: center; background-color: #FFBABA; color: #6A0E0E;"> | ||
{% trans %}This document is for an old version of Python that is no longer supported. | ||
You should upgrade, and read the {% endtrans %} | ||
<a href="/3/{{ pagename }}{{ file_suffix }}">{% trans %} Python documentation for the last stable release {% endtrans %}</a>. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest using "most recent" or "current" rather than "last" to avoid ambiguity ("most recent" vs "final").
When you're done making the requested changes, leave the comment: |
I think I'll add it thrue docsbuild-scripts, not via the file, so for future releases I won't have to commit the boolean to conf.py. |
I have made the requested changes; please review again. |
Thanks for making the requested changes! @ned-deily: please review the changes made to this pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
(cherry picked from commit 46ed90d)
Co-authored-by: Julien Palard [email protected]