Copied over latest changes from affiliated package template#64
Conversation
|
@cdeil - here's the PR with the new .travis.yml file |
There was a problem hiding this comment.
We may need to keep scipy; I think there's currently a scipy dependency somewhere in astroquery
There was a problem hiding this comment.
Yes, but it just uses numpy. I'll make a PR now removing the use of scipy.
There was a problem hiding this comment.
Scipy doesn't get picked up from apt-get anyway, it would need to be installed by pip to be picked up (as far as I can tell).
There was a problem hiding this comment.
Oh, really? That might explain some build failures I saw in the past. Does pip install scipy work on linux? I'll remove the scipy dependency right now though
|
|
|
@cdeil - done |
|
Some tests are failing because |
|
I'm sure it can be - when the code was written, there was no |
|
I'm doing it right now. |
|
@keflavich @cdeil I think we should just merge this, and then address the failures. |
|
+1 to merging this now. |
|
@keflavich - shall I merge? |
Copied over latest changes from affiliated package template
|
Failed on a threading issue: https://travis-ci.org/astropy/astroquery/jobs/6314837. Should we be concerned? I'll assume no unless subsequent builds fail with the same error |
This includes a revamped .travis.yml file