Situation
- Wir wollen die Qualität verbessern
- Fehler vermeiden
- Quereffekte erkennen
- Dokumentieren
Flash
- Wir machen CodeReviews
- fordern dafür Zeit an
- und alles wird gut... (?)
How it works
- Pre-Commit Review
- je Artefakt >> ein Review
- Verlinkungen möglich (Shop / Module)
- Kommentare/Diskussionen auf Zeilenbasis
- Ship It!
Benefits
- Code wird angesehen und hinterfragt.
- Wer macht den ersten Review ?
- Ich selbst!
- Knowledge Sharing
Limitations
- Es kann nur gereviewt werden, was editiert wurde. Vergessene Stellen werden nicht erkannt.
- Auch *intakter* Code kann Fehler verursachen
- Kein Pull-Request im SVN?
Unser Tool
- ReviewBoard
- http://reviews.united.domain
- https://www.reviewboard.org/
- Zugang nur via Ethernet oder VPN
Usecase Entwicklerwechsel
VTRACC.2451a