-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
update cextern/erfa to 1.1.0 #2497
Conversation
Should astropy include a regression test for #1031? cc @taldcroft |
[ci skip]
Oh, that's a good idea - @taldcroft, feel free to either PR against my branch of this to add said test, or as a separate PR after this is merged. |
Let's just go ahead and merge this and I'll write the regression test for #1031 after that. |
Why aren't there any Travis results or tests pending? I would like to see that before merge! |
Not sure why it didn't test the latest commit, but 41fc760 |
@taldcroft @astrofrog - There's no travis result shown on the merge button because 433f006 is the latest commit, and it has |
This updates the copy of ERFA to the just-released v1.1.0
This closes #1031 because it was fixed in the latest SOFA release.