-
Notifications
You must be signed in to change notification settings - Fork 19
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
Infrastructure updates #81
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 great, thanks. My only minor comment / question (but perhaps for a separate PR) is whether we want to start testing against a newer version of Python than 3.8
.
@moustakas, great question. Let me at least experiment with that on this PR, but if I find significant problems, I'll restore the current test suite and defer that to another PR. |
Some of the more recent versions of Astropy have different ordering when printing units, so I don't think we can test the most recent Astropy on this PR. Still the test matrix will move forward. |
I've nudged the test versions forward a little bit. Astropy > 5.0 introduces some ordering changes in printing units that will need some fixes in subsequent PRs. |
This PR makes some relatively minor updates to the package infrastructure, with the hope of not disrupting ongoing work on some filter updates, etc.
CHANGES.rst
todocs/changes.rst
so that the change log is visible on RTD.README.rst
file.