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

Update Poethepoet version #495

Merged
merged 5 commits into from
Jan 3, 2023
Merged

Update Poethepoet version #495

merged 5 commits into from
Jan 3, 2023

Conversation

Dr-Irv
Copy link
Collaborator

@Dr-Irv Dr-Irv commented Jan 3, 2023

In #249 we pinned poethepoet because Windows CI was failing as reported in #240

Based on nat-n/poethepoet#88 (comment), poethepoet has been fixed, so put the min version at 0.16.5 which fixed the issue.

@Dr-Irv Dr-Irv requested a review from bashtage January 3, 2023 16:58
@@ -217,13 +217,43 @@ class Series(IndexOpsMixin, NDFrame, Generic[S1]):
@overload
def __new__(
cls,
data: IntervalIndex[IntervalT],
data: IntervalIndex[Interval[int]],
Copy link
Member

Choose a reason for hiding this comment

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

I assume these changes are from #492

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ugh. Same mistake again. My local copy of main has the Interval change for some reason, and I keep branching off of that and making the same mistake.

This is now fixed for this PR.

@twoertwein twoertwein merged commit 5759175 into pandas-dev:main Jan 3, 2023
@twoertwein
Copy link
Member

Thanks @Dr-Irv !

@Dr-Irv Dr-Irv deleted the poethepoet branch February 4, 2023 17:05
twoertwein pushed a commit to twoertwein/pandas-stubs that referenced this pull request Apr 1, 2023
* split Series[IntervalT] into separate __new__

* update poethepoet version

* put back Series.__new__() sing IntervalT

* remove import of Interval
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