Skip to content
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

Migrate CI to GitHub Actions #178

Closed
jacobtomlinson opened this issue Nov 2, 2020 · 6 comments · Fixed by #188
Closed

Migrate CI to GitHub Actions #178

jacobtomlinson opened this issue Nov 2, 2020 · 6 comments · Fixed by #188

Comments

@jacobtomlinson
Copy link
Member

jacobtomlinson commented Nov 2, 2020

Due to changes in the Travis CI billing, the Dask org is migrating CI to GitHub Actions.

This repo contains a .travis.yml file which needs to be replaced with an equivalent .github/workflows/ci.yml file.

It also appears that this repo uses CircleCI and AppVeyor in addition to Travis. Perhaps we should consider consolidating under GitHub Actions.

See dask/community#107 for more details.

@GenevieveBuckley
Copy link
Collaborator

Thank you for the heads up!

@GenevieveBuckley
Copy link
Collaborator

GenevieveBuckley commented Nov 2, 2020

I'd say this is the TODO list for this repository:

@GenevieveBuckley
Copy link
Collaborator

It's a pity the Travis announcement didn't come a few days before the SciPy Japan sprint...

@GenevieveBuckley
Copy link
Collaborator

This comment makes a good point: dask/community#107 (comment)

Each job has options such as OS selection and the option to specify a matrix.

A note of light caution -- currently if you have a matrix of say Python versions and OS in the same workflow file, if one of those matrix entries failed you can't restart it individually, you have to trigger a full re-run of everything defined in that file. On xonsh we've subsetted them into one os per workflow file so we can at least individually restart Windows runs or MacOS runs

@jacobtomlinson
Copy link
Member Author

jacobtomlinson commented Nov 3, 2020

Thanks for adding a TODO and planning this out. I'm going to be working through other repos so happy to help out here too.

Create a scheduled action to run tests(?)

GitHub Actions makes this easy to add a new trigger which runs CI on master with a specified cron timing.

https://docs.github.com/en/free-pro-team@latest/actions/reference/events-that-trigger-workflows#schedule

However they disable cron actions if the repo hasn't been updated for 60 days. Not sure if that is what you mean by a long-ish stretch.

This comment makes a good point: dask/community#107 (comment)

Agreed it is frustrating to have to restart the whole lot, but it's more of their waste than ours. It's also an open feature request that hopefully they will get to.

@GenevieveBuckley
Copy link
Collaborator

I've been meaning to do this but never quite getting around to it.

However, the current Travis CI page is now also showing this warning (so we'll have to fix something soon): "Please be aware travis-ci.org will be shutting down in several weeks, with all accounts migrating to travis-ci.com. Please stay tuned here for more information."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants