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.

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

Replace Travis CI with GitHub Actions #137

Merged
merged 5 commits into from
Nov 25, 2020
Merged

Replace Travis CI with GitHub Actions #137

merged 5 commits into from
Nov 25, 2020

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Nov 25, 2020

Travis CI has a new pricing model which places limits on open source.

Many projects are moving to GitHub Actions instead, including Jazzband projects:

This is based on jazzband/contextlib2#26.

It drops support for Python 3.3 and 3.4 because GitHub Actions doesn't fully support them. They're EOL anyway.

Python 3.9 support is also added.

No more mention of Travis CI in the codebase:

$ git grep -i travis
$

TODO:

  • @jezdez to add JAZZBAND_RELEASE_KEY to the repo secrets.
  • @jezdez to remove continuous-integration/travis-ci and coverage/coveralls from required checks
  • @jezdez Adding GHA and Codecov as required checks after merge

@hugovk hugovk requested a review from jezdez November 25, 2020 20:06
@jezdez
Copy link
Member

jezdez commented Nov 25, 2020

@hugovk Sorry for the mess with the Jazzband implementation PR #134!

@hugovk
Copy link
Member Author

hugovk commented Nov 25, 2020

No problem, rebased!

@@ -14,7 +14,8 @@ py
pyflakes
pytest
pytest-cov
sphinxcontrib-spelling
sphinxcontrib-spelling<5.0.0; python_version == '2.7'
sphinxcontrib-spelling; python_version > '2.7'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@@ -313,7 +313,7 @@ class Counter(models.Base):
counter2 = Counter(value='2')
assert isinstance(counter2.value, int)
assert counter2.value == 2
if not six.PY3:
if six.PY2:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😬

@jezdez
Copy link
Member

jezdez commented Nov 25, 2020

I'm wondering why no action is run for this PR though?

@hugovk
Copy link
Member Author

hugovk commented Nov 25, 2020

I think it needs at least some workflow file in the repo before it's activated for the very first time.

@hugovk
Copy link
Member Author

hugovk commented Nov 25, 2020

And this is the passing build from my fork: https://github.com/hugovk/jsonmodels/actions/runs/383954700

@hugovk
Copy link
Member Author

hugovk commented Nov 25, 2020

This is the passing build from my fork: https://github.com/hugovk/jsonmodels/actions/runs/383955129

@jezdez
Copy link
Member

jezdez commented Nov 25, 2020

Gotcha, let's merge this. Thanks again @hugovk!

@jezdez jezdez merged commit 8279059 into jazzband:master Nov 25, 2020
@hugovk hugovk deleted the rm-travis branch November 25, 2020 20:41
@jezdez
Copy link
Member

jezdez commented Nov 25, 2020

I've re-enabled the checks requirement (test builds and codecov) in the master branch protection settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants