X Tutup
Skip to content

Update events handling documentation to work with Python 3.#6354

Merged
tacaswell merged 1 commit intomatplotlib:masterfrom
pganssle:event_doc_py3
May 1, 2016
Merged

Update events handling documentation to work with Python 3.#6354
tacaswell merged 1 commit intomatplotlib:masterfrom
pganssle:event_doc_py3

Conversation

@pganssle
Copy link
Copy Markdown
Member

@pganssle pganssle commented Apr 30, 2016

Event handling documentation has a lot of Python 2-style print statements, so I have updated them here. I ran the code snippets in Python 3 and they all work. The only difference is that for the Python 3 version, the outputs look like this:

'event contains', (7.0, 0.0)

And for Python 2 they look like this now:

('event contains', (7.0, 0.0))

They used to not have parens around the outputs, but they do now because of the parens required to make it polyglot code. The old behavior can return if you prepend from __future__ import print_function, but I don't really think that's necessary.

@tacaswell
Copy link
Copy Markdown
Member

👍 It looks like the docs failure is due to poor version checking in the link checker.

@tacaswell tacaswell added this to the 2.1 (next point release) milestone Apr 30, 2016
@tacaswell tacaswell closed this May 1, 2016
@tacaswell tacaswell reopened this May 1, 2016
@pganssle
Copy link
Copy Markdown
Member Author

pganssle commented May 1, 2016

Is the coverage failure messed up? It's talking about changes in coverage that didn't have anything to do with what I changed.

@tacaswell tacaswell merged commit 3b8c094 into matplotlib:master May 1, 2016
tacaswell added a commit that referenced this pull request May 1, 2016
DOC: Update events handling documentation to work with Python 3.
@tacaswell
Copy link
Copy Markdown
Member

backported to v1.5.1-doc as 520d1a1

@tacaswell
Copy link
Copy Markdown
Member

I closed and re-opened to re-trigger the builds. I do not know what it's base line is, but it might be looking at the first time it ran this PR vs the second time it ran this PR against a different master.

@QuLogic QuLogic modified the milestones: 1.5.2 (Critical bug fix release), 2.1 (next point release) May 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup