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

Use pathlib.Path in plugin hooks #193

Merged
merged 1 commit into from
Apr 13, 2024
Merged

Conversation

@michael-k
Copy link
Contributor Author

pytest 8.1 was yanked. But not because they want to revert the changes, but because the warnings did not appear properly. See pytest-dev/pytest#12069

@mr-c
Copy link

mr-c commented Mar 7, 2024

pytest 8.1 was yanked. But not because they want to revert the changes, but because the warnings did not appear properly. See pytest-dev/pytest#12069

👍 These changes are compatible with pytest 7.x+

@michael-k
Copy link
Contributor Author

Yes, my PR is compatible with both pytest 7.0.x and the yanked 8.1.0. I ran the tests in my fork and everything was green.

@carsongee carsongee merged commit 2fc6263 into carsongee:main Apr 13, 2024
@michael-k michael-k deleted the fix-192 branch April 19, 2024 08:35
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.

Crashes with pytest 8.1
3 participants