Skip to content

Clean up timefuncs tests, fix nightly due to new errors #1322

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 2 commits into from
Aug 14, 2025

Conversation

loicdiridollou
Copy link
Member

@loicdiridollou loicdiridollou commented Aug 13, 2025

  • Closes #xxxx (Replace xxxx with the Github issue number)
  • Tests added: Please use assert_type() to assert the type of any return value

Some of the new error names from pandas-dev/pandas#61468.
Nightly run is passing, see https://github.com/pandas-dev/pandas-stubs/actions/runs/16948579852.

@loicdiridollou
Copy link
Member Author

/pandas_nightly

Comment on lines 1375 to 1377
if PD_LTE_23:
# cannot shift with no freq starting in pandas 3.0.0
check(assert_type(ind.shift(1), pd.TimedeltaIndex), pd.TimedeltaIndex)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a bug in main right now, and there is a PR to fix it, so let's not make this change. See pandas-dev/pandas#62094 and the associated PR. I'm fine to live with nightly broke on this for now given it will get fixed within a few days.

@loicdiridollou
Copy link
Member Author

I will keep an eye on the PR you mentioned, also quick note I had to add # isort: skip since it kept putting the type: ignore on a new line (because line was too long) but type ignore needs to be at the from ... level

Copy link
Collaborator

@Dr-Irv Dr-Irv left a comment

Choose a reason for hiding this comment

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

@Dr-Irv Dr-Irv merged commit 0ecc8ea into pandas-dev:main Aug 14, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants