-
-
Notifications
You must be signed in to change notification settings - Fork 96
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
Make tests compatible with PyPy3.5-5.8.0 #152
Conversation
1f05a18
to
4a62d93
Compare
4a62d93
to
067f974
Compare
Codecov Report
@@ Coverage Diff @@
## master #152 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 2 3 +1
Lines 302 307 +5
=====================================
+ Hits 302 307 +5
Continue to review full report at Codecov.
|
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.
Looks good
ids=['MultiDict', 'PyMultiDict', | ||
'CIMultiDict', 'PyCIMultiDict']) | ||
@pytest.fixture( | ||
params=( |
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.
I prefer more dense formatting but can live with your notation
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.
It looked less maintainable and less readable to me when it was dense.
No description provided.