Issue34133
Created on 2018-07-17 00:51 by Nathaniel Manista, last changed 2022-04-11 14:59 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 8313 | merged | rhettinger, 2018-07-17 07:05 | |
| PR 8316 | merged | miss-islington, 2018-07-17 15:55 | |
| PR 8315 | merged | miss-islington, 2018-07-17 15:56 | |
| PR 8317 | merged | miss-islington, 2018-07-17 15:56 | |
| Messages (4) | |||
|---|---|---|---|
| msg321784 - (view) | Author: Nathaniel Manista (Nathaniel Manista) | Date: 2018-07-17 00:51 | |
The documentation for ValueError currently describes it as being "Raised when a built-in operation or function receives an argument that has the right type but an inappropriate value, and the situation is not described by a more precise exception such as IndexError.", but the Python community has (quite rightly!) adopted it as the exception to raise in any system when that system is passed a value for a parameter that is type-correct but of an invalid value. (Because what, is every library going to present a "my_library.ValueError" exception instead? That would be ridiculous.) ValueError's documentation should drop the "a built-in operation or function" wording. Perhaps go with something like "When raised indicates that a function or method was passed a value of the correct type but an invalid value"? |
|||
| msg321844 - (view) | Author: Brett Cannon (brett.cannon) * ![]() |
Date: 2018-07-17 17:41 | |
Just an FYI, Nathaniel, your title and message are bit "pushy". I've gone ahead and tweaked the title. |
|||
| msg321851 - (view) | Author: Raymond Hettinger (rhettinger) * ![]() |
Date: 2018-07-17 20:26 | |
Nathaniel, thanks for the bug report. I've fixed the docs. |
|||
| msg321854 - (view) | Author: Nathaniel Manista (Nathaniel Manista) | Date: 2018-07-17 22:54 | |
Thank you both! |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:03 | admin | set | github: 78314 |
| 2018-07-17 22:54:42 | Nathaniel Manista | set | messages: + msg321854 |
| 2018-07-17 20:26:26 | rhettinger | set | status: open -> closed resolution: fixed messages: + msg321851 stage: patch review -> resolved |
| 2018-07-17 17:41:39 | brett.cannon | set | nosy:
+ brett.cannon messages:
+ msg321844 |
| 2018-07-17 15:56:14 | miss-islington | set | pull_requests: + pull_request7852 |
| 2018-07-17 15:56:07 | miss-islington | set | pull_requests: + pull_request7851 |
| 2018-07-17 15:55:58 | miss-islington | set | pull_requests: + pull_request7850 |
| 2018-07-17 07:05:10 | rhettinger | set | keywords:
+ patch stage: patch review pull_requests: + pull_request7848 |
| 2018-07-17 06:57:21 | rhettinger | set | assignee: docs@python -> rhettinger nosy: + rhettinger |
| 2018-07-17 00:51:44 | Nathaniel Manista | create | |
