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 #397

Merged
merged 3 commits into from
Aug 31, 2021

Conversation

marvinschopf
Copy link
Contributor

Hello!

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

Travis now no longer offers unrestricted unlimited 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

@mbrukman
Copy link
Contributor

mbrukman commented Jun 2, 2021

@developit, @marvinhagemeister — could you please review this PR?

Copy link
Contributor

@mbrukman mbrukman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marvinschopf — thanks for making this change! Could you please also update the badge in README.md to point to GitHub actions instead of Travis CI?

You can use something like the following (based on what I use in my personal projects):

[![Build Status][github-ci-badge]][github-ci-url]

[github-ci-badge]: https://github.com/preactjs/preact-router/actions/workflows/node.js.yml/badge.svg
[github-ci-url]: https://github.com/preactjs/preact-router/actions/workflows/node.js.yml

.github/workflows/node.js.yml Outdated Show resolved Hide resolved
marvinhagemeister and others added 2 commits August 31, 2021 22:25
Co-authored-by: Ryan Christian <33403762+rschristian@users.noreply.github.com>
@marvinhagemeister marvinhagemeister dismissed rschristian’s stale review August 31, 2021 20:26

Added your suggestions as a commit

@marvinhagemeister marvinhagemeister merged commit a668fcc into preactjs:master Aug 31, 2021
mbrukman added a commit to mbrukman/preact-router that referenced this pull request Jan 30, 2022
This project hasn't been using Travis CI for a long time, but the README still
shows the old Travis CI badge which currently shows build status as "unknown".

This implements my earlier suggestion on the original PR to migrate from Travis
CI to GitHub Actions: preactjs#397 .
mbrukman added a commit to mbrukman/preact-router that referenced this pull request Feb 4, 2022
This project hasn't been using Travis CI for a long time, but the README still
shows the old Travis CI badge which currently shows build status as "unknown".

This implements my earlier suggestion on the original PR to migrate from Travis
CI to GitHub Actions: preactjs#397 .
mbrukman added a commit to mbrukman/preact-router that referenced this pull request Feb 4, 2022
This project hasn't been using Travis CI for a long time, but the README still
shows the old Travis CI badge which currently shows build status as "unknown".

This implements my earlier suggestion on the original PR to migrate from Travis
CI to GitHub Actions: preactjs#397 .
rschristian pushed a commit that referenced this pull request Feb 4, 2022
This project hasn't been using Travis CI for a long time, but the README still
shows the old Travis CI badge which currently shows build status as "unknown".

This implements my earlier suggestion on the original PR to migrate from Travis
CI to GitHub Actions: #397 .
happydev1237 added a commit to happydev1237/Preact-router-example that referenced this pull request Mar 29, 2023
This project hasn't been using Travis CI for a long time, but the README still
shows the old Travis CI badge which currently shows build status as "unknown".

This implements my earlier suggestion on the original PR to migrate from Travis
CI to GitHub Actions: preactjs/preact-router#397 .
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.

4 participants