-
Notifications
You must be signed in to change notification settings - Fork 3
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 installation issues #213
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, but need to see if tests pass once upstream changes are merged.
325bb86
to
e219c76
Compare
Hmm I don't think coveralls is building off both py2 and py3 versions. Can you try adding:
to the travis yml, that might make the coverage bot happy |
32901e8
to
2ea9ce9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks good to me. Consistent with installation changes in other repos too. Unless @philbull has any more comments.
Should fix the installation order issue in the same way that
pyuvdata
did recently.Also adds some installation instructions and an
environment.yml
for simpler installation of dev environment.Note that tests probably won't work until some of the upstream
hera
packages have merged their PRs for this issue.