requests / requests-oauthlib

OAuthlib support for Python-Requests!
https://requests-oauthlib.readthedocs.org/
ISC License
1.71k stars 421 forks source link

WIP: Fix Travis Pipeline #469

Closed JonathanHuot closed 2 years ago

JonathanHuot commented 2 years ago

TODO list:

JonathanHuot commented 2 years ago

Hi @nateprewitt , can you go to https://app.travis-ci.com/organizations/requests/repositories and enable Travis CI App for requests-oauthlib please ? It is required to restore TravisCI since the migration from org to com. Note that mid/long term, a migration to GH Actions can be done.

JonathanHuot commented 2 years ago

Hi @nateprewitt again, we also need to add coveralls repository, perhaps here https://coveralls.io/repos/new . I have requested access from requests community, but I'm not sure it is the right way.. wondering how it was working before. Any comments will be much appreciated. Thanks

nateprewitt commented 2 years ago

Hi @JonathanHuot ,

I've authorized Travis CI although it's prompting for CC information to use their free trial. It appears it may be working, but if not, we'll need to look into moving to Github Actions sooner than later.

In regards to Coveralls, I checked the repo and it says it's already activated. It shows as approved at the org level as well. If it's not working currently, adding codecov with Github actions is relatively straight forward with actions as well.

      - name: Upload Coverage
        uses: codecov/codecov-action@v2
        with:
          fail_ci_if_error: true
          name: "${{ runner.os }} (${{ matrix.python-version }})"
          files: ./coverage.xml
          verbose: true
jtroussard commented 2 years ago

Should we close this PR without merging? Since it sounds like we're moving to Github actions due to the potential barriers in place with Travis?

JonathanHuot commented 2 years ago

Closed in favor of https://github.com/requests/requests-oauthlib/pull/470