TYP: Enable pyright's reportInconsistentConstructor #54398
Conversation
| | ||
| class Constant(Term): | ||
| def __init__(self, value, env, side=None, encoding=None) -> None: | ||
| super().__init__(value, env, side=side, encoding=encoding) |
There was a problem hiding this comment.
Should not be needed, just calls the parent constructor (and uses value instead of name)
| DatetimeProperties, TimedeltaProperties, PeriodProperties | ||
| ): | ||
| def __new__(cls, data: Series): | ||
| def __new__(cls, data: Series): # pyright: ignore[reportInconsistentConstructor] |
There was a problem hiding this comment.
The parent __init__ expect orig as a second argument. Just ignoring the error as the comment indicates that this class is never "really" instantiated.
twoertwein changed the title | Renaming the arguments ( |
AFAICT none of the affected places are public, so I don't think thats a problem |
| exclude = ["pandas/tests", "pandas/io/clipboard", "pandas/util/version"] | ||
| # enable subset of "strict" | ||
| reportDuplicateImport = true | ||
| reportInconsistentConstructor = true |
There was a problem hiding this comment.
In a follow up, could we enable strict mode and then turn off the checks that don't pass (with todos) or are not relevant like we did with mypy?
There was a problem hiding this comment.
We could enable strict and disable many of its rules (and I think we had that in the past). I think the pyright maintainer recommended that we prioritize enabling reportGeneralTypeIssues (we have a large blacklist in pyright_reportGeneralTypeIssues.json) before enabling strict as it also changes some of its behavior.
| @@ -1,4 +1,3 @@ | |||
| # this becomes obsolete when reportGeneralTypeIssues can be enabled in pyproject.toml | |||
There was a problem hiding this comment.
Pyright uses a stricter JSON parser - comments are not part of the JSON syntax.
(There are much newer versions of pyright but there are too many things we would need to change at the moment.)
| Thanks @twoertwein |
3 participants
reportInconsistentConstructor checks that
__new__and__init__should be consistent (argument names and their types)