Remove TravisCI and Coveralls

kbandla / dpkt

fast, simple packet creation / parsing, with definitions for the basic TCP/IP protocols

Other

1.09k stars 270 forks source link

Remove TravisCI and Coveralls #515

Closed brifordwylie closed 3 years ago

brifordwylie commented 3 years ago

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.

1) Webhooks 2) Integrations 3) Branch protections

When doing 3) make sure to ADD the new GithubActions as required for merging. :)

brifordwylie commented 3 years ago

@kbandla we're probably going to start blocking on this, as it seems like the Travis queues/builds are getting a bit flaky/long.

kbandla commented 3 years ago

Deleted Webooks (TravisCI)
Deleted Integrations (TravisCI)
Branch Protection Rules
- Disabled TravisCI and Coveralls
- Enabled new GHActions for status checks (see image below)

We can delete .coveralls.yml & .travis.yml once everything is working fine.

obormot commented 3 years ago

Here is a fresh build with the latest set of checks: https://github.com/kbandla/dpkt/pull/521

Runs quick.. no more coveralls though.

brifordwylie commented 3 years ago

@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.