Skip to content

CLN: Fixing mypy errors in pandas/conftest.py #29046

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Mar 25, 2020
Merged

CLN: Fixing mypy errors in pandas/conftest.py #29046

Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
bfc175e
Ignoring some mypy errors related to imports and list inference
angelaambroz Oct 17, 2019
6408388
Adding Dtype
angelaambroz Oct 17, 2019
f4afbb2
Two spaces
angelaambroz Oct 17, 2019
ce2d9b7
MR comments
angelaambroz Oct 18, 2019
96a09ec
MR comments, removing type ignore
angelaambroz Oct 18, 2019
f07b7a0
Type Union etc
angelaambroz Oct 19, 2019
77543f2
Forgot Type
angelaambroz Oct 21, 2019
50dadb2
PR comments
angelaambroz Nov 2, 2019
5bce559
Merge conflict importing Type in _typing
angelaambroz Nov 2, 2019
1856a58
Merge remote-tracking branch 'upstream/master' into issue28926_conftest
angelaambroz Dec 13, 2019
4be05dc
PR comments
angelaambroz Dec 13, 2019
3fbf980
Update pandas/_typing.py
angelaambroz Dec 29, 2019
05bb32b
Merge conflicts
angelaambroz Jan 16, 2020
4b8b565
Fixing import sort order
angelaambroz Jan 16, 2020
a920f28
Merge conflicts
angelaambroz Mar 25, 2020
dd279e9
Linting
angelaambroz Mar 25, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion pandas/_typing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
List,
Mapping,
Optional,
Type,
TypeVar,
Union,
)
Expand Down Expand Up @@ -44,7 +45,7 @@

# other

Dtype = Union[str, np.dtype, "ExtensionDtype"]
Dtype = Union["ExtensionDtype", Type[Union[str, float, int, complex, bool]]]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Dtype = Union["ExtensionDtype", Type[Union[str, float, int, complex, bool]]]
Dtype = Union["ExtensionDtype", str, Type[Union[str, float, int, complex, bool]]]

FilePathOrBuffer = Union[str, Path, IO[AnyStr]]

# FrameOrSeriesUnion means either a DataFrame or a Series. E.g.
Expand Down
10 changes: 6 additions & 4 deletions pandas/conftest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
from decimal import Decimal
import operator
import os
from typing import List

from dateutil.tz import tzlocal, tzutc
import hypothesis
Expand All @@ -11,6 +12,7 @@
import pytest
from pytz import FixedOffset, utc

from pandas._typing import Dtype
import pandas.util._test_decorators as td

import pandas as pd
Expand Down Expand Up @@ -511,14 +513,14 @@ def tz_aware_fixture(request):

UNSIGNED_INT_DTYPES = ["uint8", "uint16", "uint32", "uint64"]
UNSIGNED_EA_INT_DTYPES = ["UInt8", "UInt16", "UInt32", "UInt64"]
SIGNED_INT_DTYPES = [int, "int8", "int16", "int32", "int64"]
SIGNED_INT_DTYPES: List[Dtype] = [int, "int8", "int16", "int32", "int64"]
SIGNED_EA_INT_DTYPES = ["Int8", "Int16", "Int32", "Int64"]
ALL_INT_DTYPES = UNSIGNED_INT_DTYPES + SIGNED_INT_DTYPES
ALL_EA_INT_DTYPES = UNSIGNED_EA_INT_DTYPES + SIGNED_EA_INT_DTYPES

FLOAT_DTYPES = [float, "float32", "float64"]
COMPLEX_DTYPES = [complex, "complex64", "complex128"]
STRING_DTYPES = [str, "str", "U"]
FLOAT_DTYPES: List[Dtype] = [float, "float32", "float64"]
COMPLEX_DTYPES: List[Dtype] = [complex, "complex64", "complex128"]
STRING_DTYPES: List[Dtype] = [str, "str", "U"]

DATETIME64_DTYPES = ["datetime64[ns]", "M8[ns]"]
TIMEDELTA64_DTYPES = ["timedelta64[ns]", "m8[ns]"]
Expand Down