Skip to content

CLN: Added static types#33169

Merged
jreback merged 6 commits into
pandas-dev:masterfrom
ShaharNaveh:CLN-StaticTypes-_libs-tslibs-parsing
Apr 3, 2020
Merged

CLN: Added static types#33169
jreback merged 6 commits into
pandas-dev:masterfrom
ShaharNaveh:CLN-StaticTypes-_libs-tslibs-parsing

Conversation

@ShaharNaveh

Copy link
Copy Markdown
Contributor
  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

I'll revert any change related to styling, I applied "black" formatting in some places, just so I could think straight.

Comment thread pandas/_libs/tslibs/parsing.pyx Outdated
object default,
bint ignoretz=False,
tzinfos=None,
dayfirst=None,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

bint no?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Doing so will require changing the default value of dayfirst, is it okay?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yes please. ping on green

Comment thread pandas/_libs/tslibs/parsing.pyx Outdated
str timestr,
object default,
bint ignoretz=False,
tzinfos=None,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

str/object

@jreback jreback added the Typing type annotations, mypy/pyright type checking label Mar 31, 2020
@jreback jreback requested a review from jbrockmendel March 31, 2020 13:42
Comment thread pandas/_libs/tslibs/parsing.pyx
@jreback jreback added this to the 1.1 milestone Mar 31, 2020

@WillAyd WillAyd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@jbrockmendel

Copy link
Copy Markdown
Member

lgtm

@WillAyd

WillAyd commented Apr 3, 2020

Copy link
Copy Markdown
Member

@jreback any outstanding comments?

@jreback jreback merged commit 07636b4 into pandas-dev:master Apr 3, 2020
@jreback

jreback commented Apr 3, 2020

Copy link
Copy Markdown
Contributor

thanks @MomIsBestFriend

@ShaharNaveh ShaharNaveh deleted the CLN-StaticTypes-_libs-tslibs-parsing branch April 6, 2020 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Typing type annotations, mypy/pyright type checking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants