Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove test-wheelhouse.sh (broken tests) #86

Closed
wants to merge 2 commits into from
Closed

Conversation

rmcgibbo
Copy link
Member

@rmcgibbo rmcgibbo commented Nov 7, 2017

No description provided.

@ehashman
Copy link
Member

ehashman commented Nov 7, 2017

@rmcgibbo rather than totally removing this, do we want to modify the Travis build matrix to make it an optional test?

@rmcgibbo
Copy link
Member Author

rmcgibbo commented Nov 7, 2017

It's not that the URLs are flaky. I think both pages are simply gone.

@matthew-brett
Copy link
Contributor

How about using the wheels at https://3f23b170c54c2533c070-1c8a9b3114517dc5fe17b7c3f8c63a43.ssl.cf2.rackcdn.com ? Those should be around for a longish time (it's the Rackspace account belonging to scikit-learn). Or pulling selected wheels from pypi?

@ehashman
Copy link
Member

ehashman commented Nov 7, 2017

@rmcgibbo yeah, as @matthew-brett suggested, I was thinking we could maybe update the URLs to use different wheels?

@njsmith
Copy link
Member

njsmith commented Nov 7, 2017

I was worried that that was going to go away because Rackspace was stopping doing free hosting for OSS, but apparently they backpedaled on that: https://www.theregister.co.uk/2017/10/20/rackspace_ends_discount_hosting_for_open_source_projects/

@ehashman
Copy link
Member

ehashman commented Nov 7, 2017

Opened #87 as an alternative to this PR per my earlier suggestion. @rmcgibbo let me know if you'd prefer that or this one!

@rmcgibbo rmcgibbo closed this Nov 8, 2017
@rmcgibbo rmcgibbo deleted the rmcgibbo-patch-1 branch November 8, 2017 06:09
@ehashman ehashman mentioned this pull request Mar 22, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants