Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-34443: Use __qualname__ instead of __name__ in enum exception messages.#14809

Merged
ethanfurman merged 2 commits into
python:masterpython/cpython:masterfrom
doerwalter:enum-qualname-exceptionsdoerwalter/cpython:enum-qualname-exceptionsCopy head branch name to clipboard
Jul 18, 2019
Merged

bpo-34443: Use __qualname__ instead of __name__ in enum exception messages.#14809
ethanfurman merged 2 commits into
python:masterpython/cpython:masterfrom
doerwalter:enum-qualname-exceptionsdoerwalter/cpython:enum-qualname-exceptionsCopy head branch name to clipboard

Conversation

@doerwalter

@doerwalter doerwalter commented Jul 17, 2019

Copy link
Copy Markdown
Contributor

@asottile asottile left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

otherwise looks good

@@ -0,0 +1,2 @@
Exceptions from :mod:`enum` now use the `__qualname` of the enum class in

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Exceptions from :mod:`enum` now use the `__qualname` of the enum class in
Exceptions from :mod:`enum` now use the ``__qualname__`` of the enum class in

@@ -0,0 +1,2 @@
Exceptions from :mod:`enum` now use the `__qualname` of the enum class in
the exception message instead of the `__name__`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
the exception message instead of the `__name__`.
the exception message instead of the ``__name__``.

@doerwalter

Copy link
Copy Markdown
Contributor Author

OK, I've fixed the ReST markup.

@ethanfurman ethanfurman merged commit 323842c into python:master Jul 18, 2019
@doerwalter doerwalter deleted the enum-qualname-exceptions branch July 19, 2019 09:57
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
…sages. (pythonGH-14809)

* Use __qualname__ instead of __name__ in enum exception messages.
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
…sages. (pythonGH-14809)

* Use __qualname__ instead of __name__ in enum exception messages.
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @doerwalter for the PR, and @ethanfurman for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-19166 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 25, 2020
…sages. (pythonGH-14809)

* Use __qualname__ instead of __name__ in enum exception messages.
(cherry picked from commit 323842c)

Co-authored-by: Walter Dörwald <walter@livinglogic.de>
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
…sages. (pythonGH-14809)

* Use __qualname__ instead of __name__ in enum exception messages.
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.

6 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.