Skip to content

Conversation

KevsterAmp
Copy link
Contributor

@KevsterAmp KevsterAmp commented Oct 5, 2025

Removed @Appender and @Substition on the following functions:

  • pandas.plotting.boxplot
  • pandas.plotting.boxplot_frame
  • pandas.plotting.PlotAccessor.line
  • pandas.plotting.PlotAccessor.bar
  • pandas.plotting.PlotAccessor.barh

@KevsterAmp
Copy link
Contributor Author

pre-commit.ci autofix

@KevsterAmp
Copy link
Contributor Author

Should I also remove the following since its not accessed by any functions after removing @Appender and @Substition on pandas/plotting/_core.py:

@mroeschke
Copy link
Member

Should I also remove the following since its not accessed by any functions after removing @Appender and @Substition on pandas/plotting/_core.py:

* https://github.com/pandas-dev/pandas/blob/b20d6ab6b6ad80ec68361321878b6a2fac272d82/pandas/plotting/_core.py#L279

* https://github.com/pandas-dev/pandas/blob/b20d6ab6b6ad80ec68361321878b6a2fac272d82/pandas/plotting/_core.py#L445

* https://github.com/pandas-dev/pandas/blob/b20d6ab6b6ad80ec68361321878b6a2fac272d82/pandas/plotting/_core.py#L454

yes

.. plot::
:context: close-figs
See Also
Copy link
Member

Choose a reason for hiding this comment

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

I don't think the indentations of the See Alsos in your PR are correct

@mroeschke mroeschke added the Docs label Oct 5, 2025
@KevsterAmp KevsterAmp requested a review from mroeschke October 6, 2025 11:53
@mroeschke mroeschke added this to the 3.0 milestone Oct 6, 2025
@mroeschke mroeschke merged commit 30cfee8 into pandas-dev:main Oct 6, 2025
42 checks passed
@mroeschke
Copy link
Member

Thanks @KevsterAmp

jzwick pushed a commit to jzwick/pandas that referenced this pull request Oct 8, 2025
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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: Replace @Appender, @Substitution, @doc with inlined docstrings
2 participants