Skip to content

Do not report plugin-generated methods with explicit-override#17433

Merged
sobolevn merged 3 commits into
masterfrom
issue-17417-2
Jun 24, 2024
Merged

Do not report plugin-generated methods with explicit-override#17433
sobolevn merged 3 commits into
masterfrom
issue-17417-2

Conversation

@sobolevn

@sobolevn sobolevn commented Jun 24, 2024

Copy link
Copy Markdown
Member

Closes typeddjango/django-stubs#2226
Closes #17417
Closes #17370
Closes #17224
This is an alternative to #17418

Thanks a lot to @sterliakov, I took a dataclasses test case from #17370

@sobolevn

Copy link
Copy Markdown
Member Author

CC @intgr and @flaeppe since this solves django-stubs issue.
CC @JukkaL in a hope for a review :)

@github-actions

This comment has been minimized.

@cdce8p

cdce8p commented Jun 24, 2024

Copy link
Copy Markdown
Collaborator

This PR looks similar (although with more tests) to #17370 which addresses the same issue.

@sobolevn

Copy link
Copy Markdown
Member Author

Thanks! Indeed, I will take a dataclasses test case from there as well.

@github-actions

Copy link
Copy Markdown
Contributor

According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

@JukkaL JukkaL left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants