search

astropy / pyregion

ds9 region parser for python
https://pyregion.readthedocs.io
MIT License
39 stars 38 forks source link

Update astropy-helpers to v2.0.6 #126

Closed bsipocz closed 6 years ago

bsipocz commented 6 years ago

This is an automated update of the astropy-helpers submodule to v2.0.6. This includes both the update of the astropy-helpers sub-module, and the ah_bootstrap.py file, if needed.

A full list of changes can be found in the changelog.

This is intended to be helpful, but if you would prefer to manage these updates yourself, or if you notice any issues with this automated update, please let @bsipocz or @astrofrog know!

Along with with the astropy core release v3.0, we have also started to release astropy-helpers v3.0.x versions. Similarly to the core package, these require Python 3.5+. We will open automated update PRs with astropy-helpers v3.0.x only for packages that specifically opt in for it when they start supporting Python 3.5+ only. Please let @bsipocz or @astrofrog know or add your package to the list in https://github.com/astropy/astropy-procedures/blob/master/update-affiliated/helpers_3.py

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 63.542% when pulling 1ec61696c0f28941046773c097cde5b538675993 on bsipocz:update-helpers-v2.0.6 into 34a90f01dde03215fd7da9e805f992d555302b46 on astropy:master.

cdeil commented 6 years ago

@bsipocz @astrofrog - with pyregion we should try to support older packages.

But I think pytest 1.7 support should probably be dropped at least: https://travis-ci.org/astropy/pyregion/jobs/345802613#L845

For the other two failing builds, I'm not sure. @bsipocz or @astrofrog - Is there a simple way to make them work? If not, how about we just drop them?

bsipocz commented 6 years ago

@cdeil - The only solution to make the numpy 1.7 test pass is to use an older astropy version with it, e.g. 1.3 or 1.0.x? Or drop it. I really do not have any preference for it and happy to do whichever way you would like.

cdeil commented 6 years ago

@bsipocz - Please drop the old Numpy. (either completely or update the version to one that is working without extra effort on your part to make it work)

bsipocz commented 6 years ago

OK, so let's merge this. If anyone feels the urge to further revise the travis matrix, please do it in a separate PR.