Skip to content

Conversation

@MSeifert04
Copy link
Contributor

@MSeifert04 MSeifert04 commented Sep 8, 2016

This may fix #741

edit:
Closes #741

@MSeifert04
Copy link
Contributor Author

🎉 😄

@keflavich
Copy link
Contributor

Ahh, did you intentionally cancel the travis-ci build? Anyway, looks good - ready to merge?

@keflavich keflavich merged commit c783770 into astropy:master Sep 8, 2016
@MSeifert04
Copy link
Contributor Author

Ahh, did you intentionally cancel the travis-ci build?

ups, no it wasn't intentional - I wanted to cancel it on my fork.

Anyway, looks good - ready to merge?

I think so. It's what they recommended in astropy/ci-helpers#128

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AppVeyor tests fail

3 participants