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 Travis tests #80

Merged
merged 2 commits into from
Sep 26, 2016
Merged

Fix Travis tests #80

merged 2 commits into from
Sep 26, 2016

Conversation

StevenMaude
Copy link
Contributor

@StevenMaude StevenMaude commented Aug 11, 2016

--use-mirrors isn't supported, so remove it.

Also, ensure we have latest setuptools, so html5lib install doesn't fail.

(html5lib has this problem fixed, just not released on PyPI yet.)

As this is not supported, and so breaks Travis tests.
@StevenMaude StevenMaude self-assigned this Aug 11, 2016
@StevenMaude
Copy link
Contributor Author

StevenMaude commented Aug 11, 2016

This is failing due to issues with html5lib whose latest version currently requires a minimum version of setuptools; I think they've fixed that recently, so this should be revisited when they release a new version.

(I think this is because messytables has unpinned html5lib as a requirement.)

@StevenMaude StevenMaude mentioned this pull request Aug 11, 2016
The current release relies on a newer version of setuptools.

They have fixed this, but haven't deployed a new release yet, so should
consider removing this fix and using a newer version when available.
@StevenMaude StevenMaude changed the title Remove --use-mirrors from pip command Fix Travis tests Sep 26, 2016
@StevenMaude StevenMaude merged commit da395c8 into master Sep 26, 2016
@StevenMaude StevenMaude deleted the remove-use-mirrors-from-pip branch September 26, 2016 10:24
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