aurzenligl / prophy

prophy: fast serialization protocol
MIT License
13 stars 8 forks source link

Add tests coverage for python 3.7-3.10 and pypy3. #35

Closed kamichal closed 2 years ago

kamichal commented 2 years ago

OK, seems that prophy works well even with python3.10 and pypy3. Just the tests needed to be adjusted a bit because of the newer clang behavior.

kamichal commented 2 years ago

Seems that Travis is not with us anymore. ...or I miss something. Where's the CI? BTW, greetings from Tampere.

florczakraf commented 2 years ago

Doesn't Travis require project owner to link the accounts and select projects to run there? @aurzenligl

Or perhaps it might be the time to migrate on to Github Actions

aurzenligl commented 2 years ago

Hey, I just visited the PR, let me check the CI :sweat_smile:

aurzenligl commented 2 years ago

It seems migration from old travis to new travis service is needed...

https://travis-ci.org/github/aurzenligl/prophy -> https://www.travis-ci.com/aurzenligl/prophy

aurzenligl commented 2 years ago

https://docs.travis-ci.com/user/billing-overview/#free-trial-plan whoops, free trial plan is available once, only for new users, and they need my credit card number. Hmm, let's explore what Github has to offer instead in terms of CI

aurzenligl commented 2 years ago

I'll try to create a PR adding support for github workflow today, or fail miserably and continue tomorrow.

aurzenligl commented 2 years ago

So far so good, it's enough to add workflow file and github automagically starts tests for PR-s: https://github.com/aurzenligl/prophy/pull/36 image I need to make this github workflow pass tests (install proper dependencies for running gtest, etc.), which I'll do tomorrow - and merge #36 before this PR can be merged. I also need to figure out how to connect github workflow coveralls.io (previously travis uploaded stuff there). Let me know what you think @kamichal and @florczakraf.

aurzenligl commented 2 years ago

I believe I merged all changes from this PR to https://github.com/aurzenligl/prophy/pull/36, let's close this one in favor of it.