Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Internal improvement: Implement GHA CI for Node 8.x, 10.x, & 12.x #2337

Merged
merged 31 commits into from Aug 29, 2019

Conversation

CruzMolina
Copy link
Contributor

@CruzMolina CruzMolina commented Aug 26, 2019

This PR leverages Github Actions to run CI checks against Node 8.x, 10.x, & 12.x on Ubuntu. Trims Travis' workload and keeps Travis jobs for Fabric EVM & Coveralls tests (for now).

Also adds Slack Notifications for when a workflow fails or when the build workflow job succeeds.

@CruzMolina CruzMolina force-pushed the testNodeBuilds branch 3 times, most recently from b8c5e9f to 289e1ec Compare August 26, 2019 23:23
@coveralls
Copy link

coveralls commented Aug 27, 2019

Coverage Status

Coverage decreased (-0.05%) to 70.498% when pulling a97659f on testNodeBuilds into 7c48d51 on develop.

@CruzMolina CruzMolina marked this pull request as ready for review August 27, 2019 17:59
@CruzMolina CruzMolina changed the title test node builds Internal improvement: Implement GHA CI for Node 8.x, 10.x, & 12.x Aug 27, 2019
@CruzMolina CruzMolina requested a review from gnidan August 27, 2019 18:03
@gnidan
Copy link
Contributor

gnidan commented Aug 28, 2019

This is really cool. Is this safe to merge? What else do we need to do? Hook up to slack?

@CruzMolina CruzMolina force-pushed the testNodeBuilds branch 2 times, most recently from 71b0a24 to e9999ed Compare August 28, 2019 16:52
@CruzMolina
Copy link
Contributor Author

@gnidan, looks like this is ready to go

Copy link
Contributor

@gnidan gnidan left a comment

Choose a reason for hiding this comment

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

nice hacking on this! let's see how it goes!

btw squash dem commits

@CruzMolina CruzMolina merged commit c22e1f7 into develop Aug 29, 2019
@CruzMolina CruzMolina deleted the testNodeBuilds branch August 29, 2019 22:48
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.

None yet

3 participants