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

Move from Travis CI to GitHub Actions #115

Merged
merged 13 commits into from May 8, 2021

Conversation

marvinschopf
Copy link
Contributor

Hello @ndresx!

I ported the Travis CI workflow to GitHub Actions in this pull request.

Travis now no longer offers unrestricted free builds for open source projects, has become very slow at times and is shutting down their travis-ci.org platform in a few weeks. So unfortunately, in my eyes, it is time to leave Travis CI behind and switch to GitHub Actions.

Kind regards
Marvin

@ndresx
Copy link
Owner

ndresx commented Feb 23, 2021

Hi @marvinschopf, thanks for the PR! I will have to look into what you did, so it will take me a while, but I think it'd be cool to also have GitHub Actions, in addition to Travis CI!

Why? First of all, they are not really shutting down their platform but are migrating it to travis-ci.com. This already works today, and also this repo has been migrated to their new environment. And the free build restriction you have mentioned only applies to their Free Plan. Open-Source projects like this will continue to be free, and even if not, the amount of builds we have here on a regular basis is very low anyway.

If you have any news that would state otherwise, please let me know. Last but not least, I would really like to try them out in parallel, just to see if it works, and how it works. So maybe you could just revert the deletion of the .travis.yml file please and as said, I'd then at some point take another look at it. Thanks again already for the initiative and your effort though!

@marvinschopf
Copy link
Contributor Author

Hello @ndresx!

I can fully understand your arguments. I took another close look at Travis' status and you are absolutely right. My apologies.

I have added the .travis.yml again. It would be really great if both services could be tried in parallel.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 82156fc on marvinschopf:github-actions into c909d97 on ndresx:master.

@ndresx ndresx merged commit ad7b612 into ndresx:master May 8, 2021
@ndresx
Copy link
Owner

ndresx commented May 8, 2021

This is now merged, thanks again!

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 this pull request may close these issues.

None yet

3 participants