Skip to content

Add CI to GitHub via Actions (based on Travis CI config)#3060

Merged
tdonohue merged 7 commits intoDSpace:mainfrom
tdonohue:github_ci
Nov 25, 2020
Merged

Add CI to GitHub via Actions (based on Travis CI config)#3060
tdonohue merged 7 commits intoDSpace:mainfrom
tdonohue:github_ci

Conversation

@tdonohue
Copy link
Copy Markdown
Member

@tdonohue tdonohue commented Nov 23, 2020

Description

As noted in Slack, Travis CI has now blocked our builds as we've exceeded the 1,000 free build minutes (based on their new policy: https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing). While we've asked for OSS plan approval, we are still waiting on approval.

Therefore, I started playing around with GitHub Actions for CI in this PR. Here's the current status of this work:

  • Create GitHub Action based on our .travis.yml.
  • Ensure code coverage is sent to Codecov
  • Delete .travis.yml

This is now ready for review/merger!

@tdonohue tdonohue added work in progress PR is still being worked on & is not currently ready for review code task Code cleanup task labels Nov 23, 2020
@tdonohue tdonohue self-assigned this Nov 23, 2020
@tdonohue tdonohue added 1 APPROVAL pull request only requires a single approval to merge. high priority and removed work in progress PR is still being worked on & is not currently ready for review labels Nov 24, 2020
@tdonohue tdonohue added this to the 7.0beta5 milestone Nov 24, 2020
@tdonohue
Copy link
Copy Markdown
Member Author

tdonohue commented Nov 25, 2020

Merging... after several small updates/tweaks, I'm now satisfied this is working properly -- the connection to Codecov now seems to be stable (namely from this commit: c7d8a9b)

NOTE: All open PRs will need to be rebased so that they include the Github Action build.yaml in their code branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge. code task Code cleanup task high priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant