Skip to content
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

[3.6] Doc: Add an optional obsolete header. (GH-13638). #13657

Merged
merged 2 commits into from Jun 15, 2019

Conversation

@JulienPalard
Copy link
Member

JulienPalard commented May 29, 2019

(cherry picked from commit 46ed90d)

Co-authored-by: Julien Palard julien@palard.fr

(cherry picked from commit 46ed90d)

Co-authored-by: Julien Palard <julien@palard.fr>
@JulienPalard

This comment has been minimized.

Copy link
Member Author

JulienPalard commented May 30, 2019

@ned-deily Hi Ned, would you please merge this, if you like it? See screenshot here: #13638

Copy link
Member

ned-deily left a comment

Are you also planning to add outdated to conf.py?

<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>.

This comment has been minimized.

Copy link
@ned-deily

ned-deily May 30, 2019

Member

Suggest using "most recent" or "current" rather than "last" to avoid ambiguity ("most recent" vs "final").

@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented May 30, 2019

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@JulienPalard

This comment has been minimized.

Copy link
Member Author

JulienPalard commented Jun 2, 2019

Are you also planning to add outdated to conf.py?

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.

@JulienPalard

This comment has been minimized.

Copy link
Member Author

JulienPalard commented Jun 15, 2019

I have made the requested changes; please review again.

@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Jun 15, 2019

Thanks for making the requested changes!

@ned-deily: please review the changes made to this pull request.

Copy link
Member

ned-deily left a comment

LGTM, thanks!

@ned-deily ned-deily merged commit 78309c9 into python:3.6 Jun 15, 2019
6 checks passed
6 checks passed
Azure Pipelines PR #20190615.23 succeeded
Details
bedevere/issue-number Issue report skipped
bedevere/maintenance-branch-pr Valid maintenance branch PR title.
bedevere/news "skip news" label found
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.