gh-107944: Improve error message for function calls with bad keyword arguments#107969
gh-107944: Improve error message for function calls with bad keyword arguments#107969pablogsal merged 1 commit intopython:mainfrom
Conversation
| Py_DECREF(suggestion_keyword); | ||
| } else { | ||
| _PyErr_Format(tstate, PyExc_TypeError, | ||
| "%U() got an unexpected keyword argument '%S'", |
There was a problem hiding this comment.
Wouldn't it be a better UX if we included a list with all the possible keywords here? We could special-case it for cases where the list is smaller than X, in order to keep the output to a reasonable size.
There was a problem hiding this comment.
I think is better, but on the other hand, it would be a bunch of extra code (joining the list with commas + handling errors) so not sure if is worth the complexity. @rhettinger what do you think? (Note we are referring to the fallback message when we could not find the suggestion either because of errors or because there is nothing close).
There was a problem hiding this comment.
what do you think?
I think what you have now will suffice — only offer a suggestion when a plausible candidate can be found; otherwise, just note that the keyword argument doesn't match.
0c61952 to
a1e2c35
Compare
…yword arguments Signed-off-by: Pablo Galindo <pablogsal@gmail.com>
|
@lysnikolaou apart from the suggestion is everything else ok with you? (In that case, could you approve the PR?) |
Uh oh!
There was an error while loading. Please reload this page.