-
Notifications
You must be signed in to change notification settings - Fork 271
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
Remove TravisCI and Coveralls #515
Comments
@kbandla we're probably going to start blocking on this, as it seems like the Travis queues/builds are getting a bit flaky/long. |
Here is a fresh build with the latest set of checks: Runs quick.. no more coveralls though. |
@obormot yeah.. coveralls was 'tied' to Travis.. Travis ran the tests which included coverage report and then sent the coverage report to coveralls. I've made a new ticket #522 for replacement functionality. @kbandla I'll go through and remove any old yml/config files.. I think there are a few other old ones laying around... you can close this ticket. |
Okay so this is now done with the merge of #514, we'll have to remove TravisCI AND Coveralls. Unfortunately I cannot do that. @kbandla will have to do it.. there are 3 places to make sure that both TravisCI and Coveralls get removed from.
When doing 3) make sure to ADD the new GithubActions as required for merging. :)
The text was updated successfully, but these errors were encountered: