For 150: updated list of implemented metrics #433
Conversation
|
This pull request #433 is assigned to @Vatavuk/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 a monetary reward for this job |
| @@ -110,7 +110,7 @@ _Revisiting Class Cohesion: An empirical investigation on several systems_,<br/> | |||
| Journal of Object Technology, vol.7, no.6, 2008, | |||
| [PDF](http://www.jot.fm/issues/issue_2008_07/article1.pdf). | |||
|
|
|||
| Here is a list of metrics we already implement: | |||
| ###Here is a list of metrics we already implement: | |||
There was a problem hiding this comment.
@paulodamaso I believe grammar is incorrect, Here is a list of metrics that we've already implemented
|
@Vatavuk @paulodamaso I didn't see you took care of this, but I got the task assigned earlier and tackled it, see #435. I believe it's more complete that this one. |
|
@paulodamaso ping |
|
@sereshqua/z please review this job completed by @Vatavuk/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
|
@Vatavuk please make sure you will find at least 3 issues during next CT @paulodamaso please make sure all issues found by code reviewers would be fixed by you if you were pull-request author again I've seen that this is no longer actual, but policy is policy |
|
@sereshqua Will do, thanks |
|
@sereshqua ok |
|
@0crat quality acceptable |
For !50: