-
Notifications
You must be signed in to change notification settings - Fork 94
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
Switch on pytest-lazy-fixtures #613
Conversation
pytest-lazy-fixture has problems with pytest 8+
Thanks for the switch! that was on our todo list, but you beat us to it :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In principle this looks good to me. Let's see what the CI says. And also, do you think we can unpin pytest?
@@ -24,7 +24,7 @@ dependencies: | |||
- coverage | |||
- pytest <8.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can pytest be unpinned now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #613 +/- ##
=======================================
Coverage 93.97% 93.97%
=======================================
Files 86 86
Lines 13811 13811
=======================================
Hits 12979 12979
Misses 832 832
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
Thanks! Merging... |
pytest-lazy-fixture has problems with pytest 8+ TvoroG/pytest-lazy-fixture#65