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

ci: use github actions #852

Merged
merged 1 commit into from
May 27, 2020
Merged

ci: use github actions #852

merged 1 commit into from
May 27, 2020

Conversation

erisu
Copy link
Member

@erisu erisu commented May 16, 2020

Motivation, Context & Description

Drop Travis CI and AppVeyor for GitHub Actions

  • Drop travis ci
  • Drop appveyor
  • Update coverage to include e2e tests
  • Update test scripts

Testing

  • npm t
  • GitHub Actions Run

Checklist

  • I've run the tests to see all new and existing tests pass

@erisu erisu added this to the 6.0.0 milestone May 16, 2020
@timbru31
Copy link
Member

Is there any way we can see if the actions will succeed once merged?

package-lock.json Outdated Show resolved Hide resolved
@NiklasMerz
Copy link
Member

Is there any way we can see if the actions will succeed once merged?

Looks like we can see the results in the fork. Seems good https://github.com/erisu/cordova-ios/actions

@erisu
Copy link
Member Author

erisu commented May 21, 2020

@timbru31 here is an example last run after the recent rebase: https://github.com/erisu/cordova-ios/actions/runs/111508020

@erisu erisu force-pushed the ci/use-gh-action branch 3 times, most recently from d76c06f to 4600e4f Compare May 27, 2020 05:38
@erisu
Copy link
Member Author

erisu commented May 27, 2020

I cleaned up the test changes and here is the latest CI run results:

https://github.com/erisu/cordova-ios/actions/runs/116486160

Copy link
Member

@NiklasMerz NiklasMerz left a comment

Choose a reason for hiding this comment

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

Still looks good

* dropped travis ci
* dropped appveyor
* added package-lock.json
* updated coverage to include e2e tests
* updated test scripts
@erisu erisu merged commit 7460737 into apache:master May 27, 2020
@erisu erisu deleted the ci/use-gh-action branch May 27, 2020 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants