Skip to content

Do not require explicit override for plugin-generated methods#17370

Closed
sterliakov wants to merge 1 commit into
python:masterfrom
sterliakov:bugfix/st-plugin-generated-override
Closed

Do not require explicit override for plugin-generated methods#17370
sterliakov wants to merge 1 commit into
python:masterfrom
sterliakov:bugfix/st-plugin-generated-override

Conversation

@sterliakov

Copy link
Copy Markdown
Collaborator

Closes #17224.

Ensure that explicit-override error is never applied to functions which are not present in source code (generated by plugin), because there is no place to apply override.

@sterliakov

Copy link
Copy Markdown
Collaborator Author

I hope that tox internal failure is not caused by my interference...

@github-actions

This comment has been minimized.

@sterliakov

sterliakov commented Jun 12, 2024

Copy link
Copy Markdown
Collaborator Author

It's transitive filelock dependency upgrade: current CI fails with 3.15.0, but succeeds with 3.14.0. I was able to reproduce this problem locally, pinning to filelock != 3.15.0 resolves the tox run problem. Would it be appropriate to add such a pin to this pull request (that breaks the "one change per PR" rule, but probably "internal dependency pin in CI" isn't that important)? Should I submit it separately (or probably investigate deeper)? Probably nothing, filelock maintainers are already engaged and work on this.

@sterliakov

Copy link
Copy Markdown
Collaborator Author

Cross-ref: tox-dev/filelock#337

@AlexWaygood AlexWaygood reopened this Jun 12, 2024
@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. ✅

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.

Subclass using @dataclass(order=True) have unavoidable "not using @override" error for __lt__ etc, with --enable-error-code=explicit-override

2 participants