Skip to content

Fix twisted test on Python3.4#427

Closed
xavfernandez wants to merge 1 commit intoprometheus:masterfrom
Polyconseil:xfernandez/fix_twisted_34
Closed

Fix twisted test on Python3.4#427
xavfernandez wants to merge 1 commit intoprometheus:masterfrom
Polyconseil:xfernandez/fix_twisted_34

Conversation

@xavfernandez
Copy link
Copy Markdown
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 xavier.fernandez@polyconseil.fr

cf
twisted/twisted@fc3be69

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

Signed-off-by: Xavier Fernandez <xavier.fernandez@polyconseil.fr>
@xavfernandez xavfernandez force-pushed the xfernandez/fix_twisted_34 branch from acd1116 to 4360a9d Compare June 13, 2019 12:12
@xavfernandez
Copy link
Copy Markdown
Contributor Author

cc @brian-brazil (with #424)

@xavfernandez
Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
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.

Comment thread tox.ini Outdated
Copy link
Copy Markdown
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
Copy Markdown
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