Skip to content

Fix nightly tests #436

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 17 commits into from
Nov 25, 2022
Merged

Fix nightly tests #436

merged 17 commits into from
Nov 25, 2022

Conversation

Dr-Irv
Copy link
Collaborator

@Dr-Irv Dr-Irv commented Nov 22, 2022

Replaces PR #433, which was a starting point

Added tests to catch exceptions

Also avoid pyright 1.1.280 because it has bugs

@Dr-Irv Dr-Irv requested a review from bashtage November 22, 2022 22:56
@Dr-Irv Dr-Irv mentioned this pull request Nov 22, 2022
2 tasks
@Dr-Irv Dr-Irv mentioned this pull request Nov 23, 2022
1 task
pyproject.toml Outdated
@@ -37,7 +37,7 @@ types-pytz = ">= 2022.1.1"
mypy = "0.990"
pyarrow = ">=9.0.0"
pytest = ">=7.1.2"
pyright = ">=1.1.278"
pyright = "<=1.1.279"
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add a pyright issue number, if it exists

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Latest main uses pyright >= 1.1.281, so this is resolved.

@Dr-Irv
Copy link
Collaborator Author

Dr-Irv commented Nov 25, 2022

@bashtage or @twoertwein this is now good to go. Can one of you merge?

@bashtage bashtage merged commit de675b7 into pandas-dev:main Nov 25, 2022
@bashtage
Copy link
Contributor

Good to have this in.

@bashtage
Copy link
Contributor

Nightly will probably be breaking a lot in the next few months unfortunately.

@Dr-Irv Dr-Irv deleted the pr433 branch December 28, 2022 15:25
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.

Nightly build appears to be broken
4 participants