Skip to content

Docs: Change Python 2 note to past tense#16651

Merged
tacaswell merged 1 commit intomatplotlib:masterfrom
hugovk:patch-2
Mar 3, 2020
Merged

Docs: Change Python 2 note to past tense#16651
tacaswell merged 1 commit intomatplotlib:masterfrom
hugovk:patch-2

Conversation

@hugovk
Copy link
Copy Markdown
Contributor

@hugovk hugovk commented Mar 3, 2020

PR Summary

PR Checklist

  • [n/a] Has Pytest style unit tests
  • [n/a] Code is Flake 8 compliant
  • [n/a] New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • [n/a] Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • [n/a] Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

I noticed the announcement on https://matplotlib.org/ is written for when 2020-01-01 is in the future:

image

The announcement for the LTS releases on https://matplotlib.org/2.2.5/index.html is:

image

This PR combines the texts, in the past tense.

@tacaswell tacaswell added this to the 3.2-doc milestone Mar 3, 2020
Copy link
Copy Markdown
Member

@timhoffm timhoffm left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

Eventually this notification box will disappear (see #16253). But it‘s good to be updated till then.

@tacaswell tacaswell merged commit cfd5463 into matplotlib:master Mar 3, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Mar 3, 2020
@hugovk hugovk deleted the patch-2 branch March 3, 2020 21:38
QuLogic added a commit that referenced this pull request Mar 4, 2020
…651-on-v3.2.0-doc

Backport PR #16651 on branch v3.2.0-doc (Docs: Change Python 2 note to past tense)
@QuLogic
Copy link
Copy Markdown
Member

QuLogic commented Mar 4, 2020

This milestone should backport to v3.2.x too, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants