search

cqfn / jpeek

Hosted and command-line calculator of cohesion metrics for Java code
https://i.jpeek.org
MIT License
205 stars 81 forks source link

SCOM.xsl:129-131: Calculate SCOM_min as per page 86 of... #142

Closed 0pdd closed 6 years ago

0pdd commented 6 years ago

The puzzle 68-03d5665d from #68 has to be resolved:

https://github.com/yegor256/jpeek/blob/aae0a5fa3342477fe23137e583903ce03c20af79/src/main/resources/org/jpeek/metrics/SCOM.xsl#L129-L131

The puzzle was created by George Aristy on 30-Jan-18.

Estimate: 30 minutes, role: IMP.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

0crat commented 6 years ago

@yegor256/z please, pay attention to this issue

0crat commented 6 years ago

Job #142 is now in scope, role is DEV

0pdd commented 6 years ago

@0pdd all puzzles are solved

0pdd commented 6 years ago

@0pdd all puzzles are solved

0crat commented 6 years ago

The job #142 assigned to @llorllale/z, here is why. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

llorllale commented 6 years ago

@0crat waiting on an answer to this question on math.stackexchange.com in order to be sure on what the mod() function means

0crat commented 6 years ago

@0crat waiting on an answer to [this](https://math.stackexchange.com/questions/2667542/understand... (here)

@llorllale The impediment for #142 was registered successfully by @llorllale/z

llorllale commented 6 years ago

@yegor256 I'm using this answer as the definition of the mod() function

llorllale commented 6 years ago

@yegor256 MetricsTest currently does not provide a way to test this. What should I do in that regard?

yegor256 commented 6 years ago

@llorllale create another ticket, I believe. Did you read this? http://www.yegor256.com/2015/01/15/how-to-cut-corners.html

llorllale commented 6 years ago

@yegor256 we already have an open issue on MetricsTest in #169 but I am not sure if I can declare it as an impediment since it's still not in scope. The proposal for #169 would allow me to add this simple test.

llorllale commented 6 years ago

@yegor256 fine, I created #222, I hope it's ok

llorllale commented 6 years ago

@0crat waiting on review for PR #223

0crat commented 6 years ago

@0crat waiting on review for PR #223 (here)

@llorllale You can't do that, unless you have one of these roles: ARC, PO. Your current roles are: DEV.

0crat commented 6 years ago

@0crat waiting on review for PR #223 (here)

@llorllale Job #142 is already on hold

0pdd commented 6 years ago

The puzzle 68-03d5665d has disappeared from the source code, that's why I closed this issue.

0crat commented 6 years ago

The job #142 is now out of scope