Skip to content

gh-144285: Improve AttributeError attribute suggestions#144299

Merged
pablogsal merged 21 commits intopython:mainfrom
johnslavik:feat-clearer-name-suggestions
Feb 12, 2026
Merged

gh-144285: Improve AttributeError attribute suggestions#144299
pablogsal merged 21 commits intopython:mainfrom
johnslavik:feat-clearer-name-suggestions

Conversation

@johnslavik
Copy link
Copy Markdown
Member

@johnslavik johnslavik commented Jan 28, 2026

Fixing up tests and adding news soon and we're ready.

I think it's okay this would affect simple cases as well. Error messages ideally are detailed and actionable.

@b9788213

This comment has been minimized.

@johnslavik johnslavik marked this pull request as ready for review February 7, 2026 19:56
@johnslavik johnslavik requested a review from pablogsal February 7, 2026 19:56
Comment thread Lib/traceback.py Outdated
@johnslavik johnslavik changed the title gh-144285: Use "instead of" technique for clearer name suggestions gh-144285: Improve AttributeError attribute suggestions Feb 7, 2026
@johnslavik
Copy link
Copy Markdown
Member Author

I caught one inconsistency. Give me a sec

@johnslavik johnslavik marked this pull request as draft February 8, 2026 15:56
@johnslavik johnslavik marked this pull request as ready for review February 8, 2026 15:58
@pablogsal
Copy link
Copy Markdown
Member

Great work @johnslavik 👌

@pablogsal pablogsal merged commit b6b72e7 into python:main Feb 12, 2026
53 checks passed
thunder-coding pushed a commit to thunder-coding/cpython that referenced this pull request Feb 15, 2026
johnslavik added a commit to johnslavik/cpython that referenced this pull request Feb 21, 2026
brijkapadia pushed a commit to brijkapadia/cpython that referenced this pull request Feb 28, 2026
brijkapadia pushed a commit to brijkapadia/cpython that referenced this pull request Feb 28, 2026
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
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.

3 participants