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 tooling change? #1513
Comments
Yes - let’s move to GitHub Actions. We discussed this in the weekly meeting on 10th dec too |
Okay, we have a workflow in now that tests with our supported Node versions on Linux and Windows. That covers most of what we were doing with Travis and Appveyor. The only other thing is coverage. Right now we run with coverage on Node 14 but don't seem to do anything with the result. |
I see https://coveralls.io/ used a lot in other open source projects. Would be a nice way to get it visible in the PR. |
@charlierudolph how do you feel about adding coveralls? |
Coverage is currently sent to coveralls but not reported on PRs. I'm good with adding coveralls and getting visibility for it on PRs |
Thanks for doing this @davidjgoss. I just disabled the appveyor project so that should stop triggering builds |
Thanks a lot for doing this @davidjgoss |
See:
Travis is looking like slightly shaky ground now.
Is it worth giving GitHub Actions a try?
cc @charlierudolph @aslakhellesoy
The text was updated successfully, but these errors were encountered: