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

Added Github action to replace travis-ci #704

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

stevengill
Copy link
Member

Summary

Travis-CI is taking to long to run and breaks easily for us. Migrating over to github actions

Requirements (place an x in each [ ])

@stevengill stevengill added the enhancement M-T: A feature request for new functionality label Dec 7, 2020
@stevengill stevengill self-assigned this Dec 7, 2020
@codecov
Copy link

codecov bot commented Dec 7, 2020

Codecov Report

Merging #704 (7c4c21f) into main (ca732ea) will decrease coverage by 2.09%.
The diff coverage is 65.68%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #704      +/-   ##
==========================================
- Coverage   84.41%   82.32%   -2.10%     
==========================================
  Files           8        8              
  Lines         693      758      +65     
  Branches      206      250      +44     
==========================================
+ Hits          585      624      +39     
- Misses         71       78       +7     
- Partials       37       56      +19     
Impacted Files Coverage Δ
src/App.ts 82.08% <65.68%> (-5.94%) ⬇️

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 7697b75...7c4c21f. Read the comment docs.

@seratch
Copy link
Member

seratch commented Dec 7, 2020

Thanks for fixing this! After merging this, the remaining TODOs are:

  • Turn off the TravisCI integration on the travis-ci.org / com side
  • Remove the TravisCI build from the required checks in branch settings

@stevengill stevengill merged commit 559984d into slackapi:main Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement M-T: A feature request for new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants