Peer Review
Opened this issue · 0 comments
Txai-HvA commented
Application of subject matter
lowerCase() & removeExtraWords()
- Comments ontbreken (een korte beschrijving bovenaan is prima) ("meer comments, meer beter")
pushWinnersIntoArray()
- Gecommente code zal ik weg halen (bijv regel 34)
- Je zou op zich een foreach kunnen gebruiken in plaats van een for loop, maar dat is voorkeur
Overig
Regel 100 t/m 116 kan je weg laten (oude gecommente code)
Quality & Process
- README is leeg
- Ik zal je wiki checken op spellingsfouten
- Goed dat je uitlegt bij je Research waarom je dingen wel of niet hebt gebruikt
- Goed dat je bij je logboek (week 2) al gaat nadenken wat je de volgende dag gaat doen
- Misschien een idee om code screenshots toe te voegen (before/after)
- Wiki moet nog aangevuld worden
- Leuk dat je bij alles een verhaaltje probeert te vertellen. Dat leest prettiger.
- Git commits hebben soms wat rare namen
- Goed dat je gebruik maakt van issues en Projects