Skip to content

Fix missing six dependency#13

Merged
UmanShahzad merged 3 commits intomasterfrom
uman/fix-deps
May 31, 2019
Merged

Fix missing six dependency#13
UmanShahzad merged 3 commits intomasterfrom
uman/fix-deps

Conversation

@UmanShahzad
Copy link
Contributor

Noticed it from ipinfo/django#3.

See commits for how it was missing.

We were doing it via installing *pytest* before,
which although worked was definitely not preferable
because it meant users would get pytest in their
prod environments.

Now we will do it by explicitly putting six in there.

Eventually we will remove python2 support and can remove six.
@UmanShahzad UmanShahzad self-assigned this May 31, 2019
@UmanShahzad UmanShahzad merged commit c3570c5 into master May 31, 2019
@UmanShahzad UmanShahzad deleted the uman/fix-deps branch May 31, 2019 21:45
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.

1 participant