Switch CI to actions

[bsipocz]

Bare minimum PR to kick-off some CI runs:

• minimalist changes to the packaging so tox is able to run
• build_docs is totally messed up locally, I run into an error, even thogh I have a recent pyvo installed. It all worked before I hacked the packaging, but atm have no time to investigate. It's very similar to automodapi crashing with sphinx 3.1.1 sphinx-automodapi#108, but I couldn't identify which dependency is missing. This is now gone

Extension error:
Handler <function process_automodsumm_generation at 0x1198247a0> for event 'builder-inited' threw an exception (exception: module 'pyvo' has no attribute 'dal')

• somehow I don't get the test output real time, it only shows up then the step is done, both on GH and also locally. Any ideas?
• remote data tests are currently ignored, those should probably go into a separate file that runs on merges, as well as for cron.

Overall, there is a ton left to do, but we should aim to modify it until we get something and then do a proper follow-up.

cc @pllim

[bsipocz]

Test are running here: https://github.com/bsipocz/astroquery/actions/

[bsipocz]

@pllim - do you have any tips why the -W for the docs build is not failing the test run? There are 17 docs warnings, all from the PRs that got merged when we didn't have any CI running (I think we shouldn't have merged anything, it's always more trouble fixing things in master rather than in the PRs.)

[bsipocz]

Also @embray and @pllim -does this config path issue rings a bell for you? I saw some possible related stuff recently in core, but you're much more in the loop atm. https://github.com/bsipocz/astroquery/runs/1649021383?check_suite_focus=true
We only run into it in oldest dependency test, which is atm astropy 3.1. I think we are totally open to the idea of bumping the min requirement to LTS if you don't have another quick solution.

[pllim]

I don't understand why the job with devastropy is also building pyregion from source.

 ERROR: Failed building wheel for pyregion

[pllim]

Ah!!! It is a Python 3.9 job. Perhaps pyregion doesn't have wheel for it.

[bsipocz]

Yes, I asked about it in the regions channel. Actually I would prefer to not have it installed than using py3.8 here. Do you know a trick of how to remove a dependency from the all list?

[pllim]

Do you know a trick of how to remove a dependency from the all list?

Unfortunately, I don't. The only options I know of are either refactoring the listing in setup.cfg or manually state your dependencies in tox.ini.

[pllim]

somehow I don't get the test output real time,

Depends on GitHub Actions buffering, I am guessing. I don't think it is within our control. Unless I misunderstood?

build_docs -W

I see you are still using python setup.py build_sphinx -W, which is no longer supported after APE 17. I would recommend deferring to APE 17 transition to fix, or use RTD PR builder with the option to turn warning into failure.

[bsipocz]

Oldest astropy needed to be bumped to 3.1.2, but with that everything seems to work. One more rebase, and if this keep passing, I'm merging it.

I'm opening an issue for the still outstanding stuff.

[bsipocz]

And the end I also needed to remove aplpy as well as pyregion from the dependencies for py39 as the former relies on shapely that doesn't build.

[bsipocz]

We have CI yet again! 🎉

[pllim]

Re: skipping CI (saw your reply in email but cannot find it here no more) -- Yes, you are welcome to use https://github.com/pllim/action-skip-ci/ . I deployed it in several of my projects and it works for me. For example: https://github.com/spacetelescope/synphot_refactor/blob/master/.github/workflows/ci_workflows.yml

[bsipocz]

Yes, it's in the collapsed conversation above, I've noticed that the links in emails/notifications not always work properly :(