-
Notifications
You must be signed in to change notification settings - Fork 314
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
Remove Codecov on master and production branches #3743
Remove Codecov on master and production branches #3743
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@itstrueitstrueitsrealitsreal is attempting to deploy a commit to the modsbot's projects Team on Vercel. A member of the Team first needs to authorize it. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3743 +/- ##
=======================================
Coverage 53.87% 53.87%
=======================================
Files 274 274
Lines 6032 6032
Branches 1449 1449
=======================================
Hits 3250 3250
Misses 2782 2782 ☔ View full report in Codecov by Sentry. |
Thank you for your contribution! Just wanted to let you know that we haven't forgotten about your PR, will get to it hopefully this weekend :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for contributing! LGTM 😊
Context
Fixes #3708
Implementation
Adds a conditional statement in
.circleci/config.yml
to check if the current branch ismaster
orproduction
, and does not runyarn codecov --disable=gcov -f ./coverage/coverage-final.json
if so. This reduces build times for both of these branches.