TYP: annotation of __init__ return type (PEP 484) (pandas/plotting) #46283
Conversation
| _DEFAULT_KEYS = ["xaxis.compat"] | ||
|
|
||
| def __init__(self, deprecated=False): | ||
| def __init__(self, deprecated=False) -> None: |
There was a problem hiding this comment.
as long as you're doing these can you try to annotate other things in these signatures
There was a problem hiding this comment.
No problem, I think I can do it in separate PR as long as these are for fixing __init__ return type annotation
There was a problem hiding this comment.
yeah, let's maybe keep the scope here limited to the return type of __init__ since if there is consensus on #46337, there will be quite a bit to do (if not done in one big PR, #46337 (comment))
| _all_kinds = _common_kinds + _series_kinds + _dataframe_kinds | ||
|
|
||
| def __init__(self, data): | ||
| def __init__(self, data) -> None: |
There was a problem hiding this comment.
I am pretty sure that not annotating this was a deliberate decision in the past.
cc @simonjayhawkins did this change over the last few weeks?
There was a problem hiding this comment.
I am pretty sure that not annotating this was a deliberate decision in the past.
cc @simonjayhawkins did this change over the last few weeks?
As far as I can see, this line was added on 03/07/2019(#27009) without any additional comments. And this line also has not changed since those times.
There was a problem hiding this comment.
I am talking about init returning None in general
|
thanks @nafarya FYI we are going to do these in a few PRs that cover most of the codebase |
5 participants
According to documentation pandas uses PEP484 for type hints.
PEP484 states that
__init__function should have-> Nonereturn type annotation.In this PR I added return type annotation to
__init__functions inpandas/plottingmodule.Thanks!