Skip to content

DOC: make return type documentation of series methods consistent #35409 #36862

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

Merged
merged 8 commits into from
Oct 10, 2020

Conversation

junjunjunk
Copy link
Contributor

@junjunjunk junjunjunk commented Oct 4, 2020

@junjunjunk
Copy link
Contributor Author

junjunjunk commented Oct 4, 2020

Fix docstrings associated with Series methods for now.
Please review.

Once it passes review, I'll start working on other docstring fixes in the same way.

@@ -917,7 +916,8 @@ def reorder_categories(self, new_categories, ordered=None, inplace=False):

Returns
-------
cat : Categorical with reordered categories or None if inplace.
cat : Categprocal or None
Copy link
Member

Choose a reason for hiding this comment

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

Typo here. Otherwise, lgtm

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you. Done.
I start working on other docstring fixes in the same way.

Copy link
Member

@ivanovmg ivanovmg Oct 4, 2020

Choose a reason for hiding this comment

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

Note that CI Checks complains about missing dots at the end of return statement.

6818:RT05:pandas.DataFrame.interpolate:Return value description should finish with "."

There are several of such errors.

Copy link
Contributor Author

@junjunjunk junjunjunk Oct 4, 2020

Choose a reason for hiding this comment

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

I'm sorry for all the mistakes. Thank you for pointing that out. Done.

@rhshadrach rhshadrach added the Docs label Oct 5, 2020
@junjunjunk
Copy link
Contributor Author

junjunjunk commented Oct 7, 2020

would love to get a hacktoberfest tag if possible. #36837

@jreback jreback added this to the 1.2 milestone Oct 10, 2020
@jreback
Copy link
Contributor

jreback commented Oct 10, 2020

cc @simonjayhawkins ok with the text?

@simonjayhawkins simonjayhawkins merged commit b110046 into pandas-dev:master Oct 10, 2020
@simonjayhawkins
Copy link
Member

Thanks @junjunjunk! Very nice.

@junjunjunk junjunjunk deleted the is35409 branch October 15, 2020 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: Inconsistent return type documentation for functions with "inplace" option
5 participants