-
Notifications
You must be signed in to change notification settings - Fork 170
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
(#1294) Removed unnecessary collection HeadOf #1310
Conversation
This pull request #1310 is assigned to @victornoel/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job |
Codecov Report
@@ Coverage Diff @@
## master #1310 +/- ##
============================================
- Coverage 89.57% 89.56% -0.02%
+ Complexity 1671 1669 -2
============================================
Files 284 283 -1
Lines 4008 4004 -4
Branches 211 211
============================================
- Hits 3590 3586 -4
Misses 385 385
Partials 33 33 Continue to review full report at Codecov.
|
@paulodamaso it's good to merge |
@rultor merge |
@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here |
@paulodamaso Done! FYI, the full log is here (took me 17min) |
@sereshqua/z please review this job completed by @victornoel/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
Payment to |
@0crat quality bad |
Quality is low, no payment, see §31 |
Quality review completed: +4 point(s) just awarded to @sereshqua/z |
This PR is for #1294