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

Fix twisted test on Python3.4 #427

Closed

Conversation

xavfernandez
Copy link
Contributor

@xavfernandez xavfernandez commented Jun 13, 2019

cf
twisted/twisted@fc3be69
and twisted/twisted@8a47339 broke Python3.4 between 19.2.0 and 19.2.1

Signed-off-by: Xavier Fernandez [email protected]

cf
twisted/twisted@fc3be69

and
twisted/twisted@8a47339
broke Python3.4 between 19.2.0 and 19.2.1

Signed-off-by: Xavier Fernandez <[email protected]>
@xavfernandez
Copy link
Contributor Author

cc @brian-brazil (with #424)

@xavfernandez
Copy link
Contributor Author

This fixes the failing test of master branch: https://travis-ci.org/prometheus/client_python/builds/542696662

Copy link
Contributor

@brian-brazil brian-brazil left a comment

Choose a reason for hiding this comment

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

Oops, forgot to press Submit on my review.

tox.ini Outdated
@@ -29,7 +29,8 @@ deps =
[testenv]
deps =
{[base]deps}
{py27,py34,py35,py36,pypy,pypy3}: twisted
{py27,py35,py36,pypy,pypy3}: twisted
{py34}: twisted==19.2.1
Copy link
Contributor

Choose a reason for hiding this comment

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

We're only meant to test everything on the latest of each minor version, so you can drop 3.4 and 3.5

Copy link
Contributor Author

@xavfernandez xavfernandez Jun 17, 2019

Choose a reason for hiding this comment

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

Is #428 what you meant ?

@xavfernandez xavfernandez deleted the xfernandez/fix_twisted_34 branch June 17, 2019 08:34
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.

2 participants