Perform manual code inspection summary

0 votes
asked Oct 17, 2017 in SWSV by uknownothing (28 points)  

A reviewt azt minden csapattag végzi ez egyértelmű.
A kérdés a következő pontra irányulna:
" Write a short summary (2-3 paragraphs) about the functionality and responsibility of the class under review (SafetyLogicImpl)."

Itt nem volt kiemelve, hogy ezt egy valaki csinálja, mint pár sorral lejjebb, de jól értelmezzük, hogy nem kell mindenkinek summaryt írnia róla, vagy tényleg mindenkinek kell összefoglalót írni az osztály funkcionalitásáról?

1 Answer

+1 vote
answered Oct 17, 2017 by hajdua (688 points)  
selected Oct 17, 2017 by micskeiz
 
Best answer

Valóban nem egyértelmű az issue szövege. Az aspektusok definiálását és az összefoglalót lehet közösen készíteni, csak maga a review az, amit mindenkinek el kell végeznie. Ez viszont jó lenne, ha látszódna is, tehát például amikor összegyűjtitek a megtalált problémákat, akkor az legyen lebontva személyenként. Vagy akár az is egy jó megoldás, ha az issue alá mindenki odaírja kommentben az általa megtalált problémákat, majd ebből állítjátok össze a végleges listát.

...