-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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.
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
Changed 'int' type to 'integer' in to_numeric docstring #35776
Changed 'int' type to 'integer' in to_numeric docstring #35776
Conversation
Thanks @edwardkong. The docstring for SparseArray also seems to've been changed in error by the same PR, can you fix that here also? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @edwardkong for the PR. lgtm ex @dsaxton comment
can you also revert the changes to pandas/core/arrays/sparse/array.py made in #33093
i'll add this to the 1.1.1 milestone as effectively a regression in the docs. feel free to change if disagree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Thanks @edwardkong Since I want to make sure this is fixed for 1.1.1 I pushed this change. |
@jreback ok to merge (and backport) |
…umeric docstring
Thanks @edwardkong |
…string (#35806) Co-authored-by: edwardkong <[email protected]>
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff