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

Pin to pytest==7.* #1020

Merged

Conversation

pentschev
Copy link
Member

Pytest 8 introduced breaking changes that pytest-asyncio is still in the process of resolving. See
pytest-dev/pytest-asyncio#763. This pin should also be done directly by pytest-asyncio>=0.23.4, but that version is still not available in conda-forge.

Backport of #1019 .

Pytest 8 introduced breaking changes that `pytest-asyncio` is still in the
process of resolving. See
pytest-dev/pytest-asyncio#763. This pin should
also be done directly by `pytest-asyncio>=0.23.4`, but that version
is still not available in conda-forge.
@pentschev pentschev changed the base branch from branch-0.37 to branch-0.36 January 30, 2024 13:08
@pentschev pentschev added bug Something isn't working non-breaking labels Jan 30, 2024
Copy link
Member

@charlesbluca charlesbluca left a comment

Choose a reason for hiding this comment

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

Thanks @pentschev! Looks like the issue you linked is closed but progress on the hotfix is being tracked in pytest-dev/pytest-asyncio#737.

Also worth noting that it looks like the hotfix is actually released now, and we're just waiting on the conda-forge release PR to get merged:

conda-forge/pytest-asyncio-feedstock#40

@pentschev
Copy link
Member Author

Yup, I saw the conda-forge PR but while it isn't merged we can't count on it, so best to just go ahead and pin it ourselves in the meantime. Thanks @charlesbluca !

@pentschev
Copy link
Member Author

/merge

@raydouglass raydouglass merged commit d79b7e9 into rapidsai:branch-0.36 Jan 30, 2024
33 checks passed
@pentschev pentschev deleted the pin-pytest-7-branch-0.36 branch January 30, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants