Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

Gets tests passing before I starting adding support for virtualenv 20+ #234

Merged
merged 6 commits into from
Jan 28, 2021

Conversation

analogue
Copy link
Contributor

make test is green (on bionic at least).

Comment on lines -66 to -69
# old setuptools and old pip
['setuptools==0.6c11', 'pip==1.4.1'],
# old setuptools and new pip
['setuptools==0.6c11', 'pip==1.5.6'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Another "fails on Python2 and not worth figuring out why" unless someone has other opinions.

@analogue
Copy link
Contributor Author

Travis is taking forever - builds sitting around for hours waiting. Gonna see if I can knock out #235

@analogue
Copy link
Contributor Author

analogue commented Jan 28, 2021

Travis is taking forever - builds sitting around for hours waiting. Gonna see if I can knock out #235

TL;DR went down a rabbit hole with some weird failure (see #236 ) Abandoning for now.

Back to this PR, tested locally and make test is green: https://fluffy.yelpcorp.com/i/jj3KnhPM7wcZRJwKbdKsjTdFkBBMQlTZ.html

Copy link
Member

@ymilki ymilki left a comment

Choose a reason for hiding this comment

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

Thanks. Looks like this needs a little love so it passes tests everywhere we want. Locally passing is a good start.

@analogue analogue merged commit 150e761 into master Jan 28, 2021
@analogue analogue deleted the spatel/muck branch January 28, 2021 19:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants