Skip to content

Conversation

jbrockmendel
Copy link
Member

Nothing serious: improve some typing, de-duplicate, hopefully-helpful comments, remove a couple unreachable paths.

@mroeschke mroeschke added this to the 3.0 milestone Sep 24, 2025
@mroeschke mroeschke merged commit a072875 into pandas-dev:main Sep 24, 2025
51 of 52 checks passed
@mroeschke
Copy link
Member

Thanks @jbrockmendel

@jorisvandenbossche
Copy link
Member

Some typing errors because of this:

  /home/runner/work/pandas/pandas/pandas/plotting/_matplotlib/converter.py:236:31 - error: Cannot access attribute "freq" for class "Axis"
    Attribute "freq" is unknown (reportAttributeAccessIssue)
  /home/runner/work/pandas/pandas/pandas/plotting/_matplotlib/converter.py:280:35 - error: Cannot access attribute "ordinal" for class "NaTType"

They were hidden because of the doctest failure that is present on main at the moment (I am fixing that)

@jbrockmendel jbrockmendel deleted the ref-plotting branch September 24, 2025 21:36
@jbrockmendel jbrockmendel mentioned this pull request Sep 24, 2025
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.

3 participants