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.

Sign up for GitHub

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

Jump to bottom

Drop Travis as our CI provider #272

Merged
merged 3 commits into from
Sep 4, 2021
Merged

Drop Travis as our CI provider #272

merged 3 commits into from
Sep 4, 2021

Commits on Sep 4, 2021

  1. Drop Travis as our CI provider 

    Since travis changed its pricing policy it has become more limited in
what we can accomplish with it. It now uses a limited credit model for
open source projects which besides being cumbersome puts the project in
a precarious position where we may be unable to run tests if we dont
have anymore credits.

For this reason we will be moving to github actions since they seem to
be the best alternative at the moment
    @alan-barzilay
    alan-barzilay committed Sep 4, 2021
    Configuration menu
    Copy the full SHA
    92b9dd7 View commit details
    Browse the repository at this point in the history

  2. Add flake8 github action with review dog 

    Review dog is an incredible project that makes linting and formatting
review a breeze by commenting inline what is wrong in a pull request.
This makes the review process easier for the maintainer and also
provides a clearer feedback to the contributor
    @alan-barzilay
    alan-barzilay committed Sep 4, 2021
    Configuration menu
    Copy the full SHA
    5689124 View commit details
    Browse the repository at this point in the history

  3. Add tox tests & codecov github action 

    This commit essentially adds back tests to our CI pipeline. They were
previously dropped due to Travis pricing policy change.

This workflow utilizes a few interesting projects to make this action
easier to maintain such as the codecov github action
and the tox-gh-actions project
(https://github.com/ymyzk/tox-gh-actions)

This commit uses codecov instead of coveralls because using coveralls
directly inside GH-actions is buggy and the official coveralls action
only supports lcov reports which we can't seem to be able to generate at
the moment. For more information see the pull request that introduced
this commit
    @alan-barzilay
    alan-barzilay committed Sep 4, 2021
    Configuration menu
    Copy the full SHA
    bc8121e View commit details
    Browse the repository at this point in the history