Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

[develop] Github Actions for Community PRs #8411

Merged
merged 8 commits into from
Jan 14, 2020
Merged

Conversation

scottarnette
Copy link
Contributor

@scottarnette scottarnette commented Jan 8, 2020

Change Description

Problems:

  • Travis limits us to a certain amount of jobs at a time for the entire github org.
  • It is also limited on total time a job can run.
  • The hardware available is limited and slow.

Solution:

  • Replace Travis with github actions so that we can triple the amount of concurrent jobs.
  • Github actions job timeouts are much longer than Travis.
  • The hardware is far more powerful in github actions.

See:
https://github.com/scottarnette/eos/pull/3 | Forked PR. This PR would merge this branch into my fork of EOS.

https://github.com/scottarnette/eos/runs/381994338 | Actions build using commit 77d6721 of this branch on my fork.

https://github.com/EOSIO/eos/commit/51dd4a8b9a44874cf3640a2926d8ccd309b3cce0/checks?check_suite_id=390896314 | Actions build of this PR. Because this PR is not a fork, we do not run build steps on Actions.

NOTE: If all steps in Actions are skipped, the build is marked as failed. The "Start Job" step exists to fix this issue. All build steps are triggered from that step IF the PR is forked.

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@nksanthosh nksanthosh requested a review from arhag January 14, 2020 16:18
@NorseGaud NorseGaud changed the title [develop] Actions [in progress] [develop] Github Actions for Community PRs Jan 14, 2020
@NorseGaud NorseGaud merged commit 6e7df98 into develop Jan 14, 2020
@heifner heifner deleted the cicd-actions-dev branch March 12, 2020 12:37
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