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

Replace Travis CI with GitHub Actions #820

Merged
merged 3 commits into from
Feb 3, 2021
Merged

Replace Travis CI with GitHub Actions #820

merged 3 commits into from
Feb 3, 2021

Conversation

hugovk
Copy link
Contributor

@hugovk hugovk commented Feb 3, 2021

Travis CI has a new pricing model which places limits on open source.

Travis support has also become unresponsive to developers who have run out of free credits, and many projects are moving to GitHub Actions instead.

And https://github.com/pypa builds can not longer build:

Builds have been temporarily disabled for public repositories due to a negative credit balance. Please go to the Plan page to replenish your credit balance or alter your Consume paid credits for OSS setting.

image

Owner pypa does not have enough credits.

image


This GHA config does much the same as the deleted Travis one. Two differences: tests with current newest Python 3.9 instead of 3.6; and adds caching of pip downloads, to try and help ease the load on PyPI, and speed up builds a bit.

Sample build:

.github/workflows/test.yml Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
@webknjaz
Copy link
Member

webknjaz commented Feb 3, 2021

FWIW this PR isn't urgent because we have an RTD-based check in PRs that essentially does the same smoke-test. Other than that, I fully support this effort.

@webknjaz webknjaz added type: task Something that needs to be done that is not a bug or feature type: enhancement A self-contained enhancement or new feature labels Feb 3, 2021
@webknjaz webknjaz requested review from di and pradyunsg February 3, 2021 14:06
@di di merged commit 212f0c1 into pypa:main Feb 3, 2021
@hugovk hugovk deleted the rm-travis branch February 3, 2021 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A self-contained enhancement or new feature type: task Something that needs to be done that is not a bug or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants