Skip to content
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

DOC: Resolve RT03 errors in several methods #2 #57785

Merged
merged 8 commits into from
Mar 8, 2024

Conversation

bergnerjonas
Copy link
Contributor

Resolve all RT03 errors for the following cases:
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.nsmallest
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.nunique
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.pipe
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.plot.box
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.plot.density
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.plot.kde
scripts/validate_docstrings.py --format=actions --errors=RT03 pandas.DataFrame.plot.scatter

@bergnerjonas bergnerjonas changed the title Resolve RT03 errors in several methods #2 DOC: Resolve RT03 errors in several methods #2 Mar 8, 2024
@mroeschke mroeschke added this to the 3.0 milestone Mar 8, 2024
@mroeschke mroeschke added the Docs label Mar 8, 2024
@mroeschke mroeschke merged commit 4692686 into pandas-dev:main Mar 8, 2024
48 checks passed
@mroeschke
Copy link
Member

Thanks @bergnerjonas

pmhatre1 pushed a commit to pmhatre1/pandas-pmhatre1 that referenced this pull request May 7, 2024
…57785)

* Add return info for nsmallest method

* Add result info on nunique.

* Add return information to pipe method.

* Add return information for boxplot method

* Add return information for kde plot.

* Add result information for scatter plot.

* Remove resolved methods from ignore_functions parameter

* Reinsert method to code checker which was erroneously removed.
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.

2 participants