Skip to content
This repository was archived by the owner on Jan 12, 2023. It is now read-only.

Migrate from Travis to Github Actions - #139

Merged
c-w merged 4 commits into
masterfrom
github-actions
Nov 27, 2020
Merged

Migrate from Travis to Github Actions#139
c-w merged 4 commits into
masterfrom
github-actions

Conversation

@c-w

@c-w c-w commented Nov 27, 2020

Copy link
Copy Markdown
Owner

Queue times on Travis have been steadily rising so moving over to Github Actions to speed up CI. Also switching PyPI authentication to use an API token instead of a bot password to scope down access rights of the release account.

@MasterOdin MasterOdin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also good to move off Travis now after they made their OSS pricing changes so all free repos get 1000 minutes that they then need to ask more for (unclear if it resets every month, or if it was 1000 minutes in total).

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Co-authored-by: Matthew Peveler <matt.peveler@gmail.com>
@codecov-io

codecov-io commented Nov 27, 2020

Copy link
Copy Markdown

Codecov Report

Merging #139 (38f8dfa) into master (6a89a69) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #139   +/-   ##
=======================================
  Coverage   97.22%   97.22%           
=======================================
  Files          28       28           
  Lines        1080     1080           
=======================================
  Hits         1050     1050           
  Misses         30       30           

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 6a89a69...d8f8e49. Read the comment docs.

c-w and others added 2 commits November 27, 2020 08:11
Co-authored-by: Matthew Peveler <matt.peveler@gmail.com>
@c-w
c-w requested a review from MasterOdin November 27, 2020 13:22
@c-w
c-w merged commit ee0ba62 into master Nov 27, 2020
@c-w
c-w deleted the github-actions branch November 27, 2020 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants