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

Migrate from Travis to GitHub actions #43

Merged
merged 6 commits into from
Dec 21, 2020
Merged

Migrate from Travis to GitHub actions #43

merged 6 commits into from
Dec 21, 2020

Conversation

aleksihakli
Copy link
Member

@aleksihakli aleksihakli commented Dec 16, 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.

TODO:

  • @jezdez to add JAZZBAND_RELEASE_KEY to the repo secrets.

@aleksihakli
Copy link
Member Author

@jezdez opened a PR for django-ical in the same run, this supposedly also needs an updated PyPI mirror key?

@jezdez jezdez added this to In progress in Migrate to GitHub Actions via automation Dec 16, 2020
@jezdez
Copy link
Member

jezdez commented Dec 16, 2020

@jezdez opened a PR for django-ical in the same run, this supposedly also needs an updated PyPI mirror key?

Yes, could you please use the PR text as I've used in the other Jazzband projects? E.g. jazzband/django-pipeline#738

If you add me to the PR text it allows me track the progress better.

@jezdez
Copy link
Member

jezdez commented Dec 16, 2020

FTR I have more time again later this week to work on those ports and will get back to you then withr reviews

@codecov
Copy link

codecov bot commented Dec 16, 2020

Codecov Report

Merging #43 (6c59a8f) into master (e9c7909) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   96.97%   96.97%           
=======================================
  Files           5        5           
  Lines         695      695           
  Branches       43       43           
=======================================
  Hits          674      674           
  Misses          9        9           
  Partials       12       12           
Impacted Files Coverage Δ
django_ical/feedgenerator.py 96.87% <100.00%> (ø)
django_ical/tests/test_feed.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f2bcc2...06550f6. Read the comment docs.

@aleksihakli
Copy link
Member Author

Sure thing, I'll use the full project and issue IDs in any upcoming PRs.

@jezdez jezdez moved this from In progress to In review in Migrate to GitHub Actions Dec 18, 2020
Migrate to GitHub Actions automation moved this from In review to Reviewed Dec 21, 2020
@jezdez jezdez merged commit 595e4c3 into master Dec 21, 2020
Migrate to GitHub Actions automation moved this from Reviewed to Done Dec 21, 2020
@jezdez jezdez deleted the ci branch December 21, 2020 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants