-
-
Notifications
You must be signed in to change notification settings - Fork 44
rename _eva_ to apyhtest_eva #230
Conversation
|
Weird, but okay. Anyone have any idea what's wrong with the CI? |
|
Not sure, it seems to be related to a heading underline being the wrong length, presumably because of the package name changing length? |
|
Oops, yeah, my bad. Will fix that and then rebase |
|
(and note that this isn't getting a milestone because it's a testing-only change) |
|
OK, this is now passing (the appveyor issue seems to be due to graphviz's web site being down...). @embray and/or @astrofrog, look good to you? |
|
👍 |
|
Alright - merging! |
|
@astrofrog - I've noticed that v1.0.7 is failing. Could you please backport this one, it takes care some of the failures. (Locally only one remained when I've cherry-picked these commits). |
|
Ok, so the branch on top of v.1.0.7 with the cherry-picked commits from various PR is this: https://travis-ci.org/bsipocz/astropy-helpers/builds/136710524 |
rename _eva_ to apyhtest_eva
rename _eva_ to apyhtest_eva
This is an effort to address the problem in #222 based on @astrofrog's #222 (comment) .
I'm actually having some trouble reproducing the issue locally - it may be that setuptools 21.x has fixed it? It's probably good to have this in for insurance anyway, though...
At any rate, let's see if the tests pass. If so, then I think this closes #222.