8000 TYPING: add some type hints to core.generic by simonjayhawkins · Pull Request #27646 · pandas-dev/pandas · GitHub
[go: up one dir, main page]
Skip to content

TYPING: add some type hints to core.generic #27646

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

Closed
wants to merge 22 commits into from
Closed

TYPING: add some type hints to core.generic #27646

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
05dcd23
TYPING: add some type hints to core.generic
simonjayhawkins Jul 29, 2019
3dfecd1
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Jul 29, 2019
a207010
black
simonjayhawkins Jul 29, 2019
978029e
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Jul 31, 2019
3ba8c76
move typing_extensions to _typing.py
simonjayhawkins Jul 31, 2019
9aa69fe
add whatsnew
simonjayhawkins Jul 31, 2019
4346a6b
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Aug 3, 2019
8fb0254
use Literal for Axis Alias
simonjayhawkins Aug 3, 2019
3b04221
return type Type[FrameOrSeries] for _constructor
simonjayhawkins Aug 3, 2019
ec609ee
resolve merge conflict
simonjayhawkins Aug 5, 2019
2f35f26
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Aug 5, 2019
ac31e7d
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Aug 6, 2019
7c4e87a
remove typing_extensions
simonjayhawkins Aug 6, 2019
f56794d
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Aug 8, 2019
0ce386e
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Aug 24, 2019
b8195b9
resolve merge conflicts
simonjayhawkins Aug 27, 2019
3690f0b
remove extra line
simonjayhawkins Aug 27, 2019
416c461
resolve merge conflict
simonjayhawkins Aug 28, 2019
aa71327
8000 import FrameOrSeries and _T from _typing
simonjayhawkins Aug 28, 2019
6397422
lint error - unused import
simonjayhawkins Aug 28, 2019
b8866d6
revert changes to runtime behaviour
simonjayhawkins Aug 29, 2019
90833db
Merge remote-tracking branch 'upstream/master' into type-hints-core.g…
simonjayhawkins Aug 29, 2019
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
Prev Previous commit
Next Next commit
remove extra line
  • Loading branch information
@simonjayhawkins
simonjayhawkins committed Aug 27, 2019
commit 3690f0b28ac9898c24e7ae4f0bbfccadaf1ec34d
1 change: 0 additions & 1 deletion pandas/core/generic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@
from pandas.io.formats.printing import pprint_thing
from pandas.tseries.frequencies import to_offset


if TYPE_CHECKING:
from pandas import Series, DataFrame
Copy link
Contributor

Choose a reason for hiding this comment

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

I would put these in _typing

Copy link
Member Author

Choose a reason for hiding this comment

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

the errors is local to this module since I think a global errors should also include coerce?

similarly a global How should probably be left, right, inner, outer?

I could rename them if you think these have additional utility, to say IgnoreRaise and FirstLast


Expand Down
0