Commit 9b166706 authored by Lukas Geiter's avatar Lukas Geiter

Add more review details

parent 01c160ed
......@@ -624,9 +624,11 @@ Visualisiert Abhängigkeiten mit Richtung und Anzahl Referenzen.
\section{Technical Debt}
Code Smells, Fehlende Tests, Q-Mängel, Keine Zeit für Review, etc.
\section{Code Reviews}
\section{Reviews}
Immer zwischen Peers aus der eigenen Abteilung mit Gruppe mehrerer Personen, definierten Rollen, definiertem Stück Code, begrenzter Zeit, Protokoll und Nacharbeiten.
\subsection{Code Reviews}
Immer zwischen Peers aus der eigenen Abteilung mit Gruppe mehrerer Personen, definierten Rollen, definiertem Stück Code (100-500 LOC), begrenzter Zeit (2-4h), Protokoll und Nacharbeiten.
\begin{enumerate}
\item Frühe Fehlerentdeckung
......@@ -635,6 +637,19 @@ Immer zwischen Peers aus der eigenen Abteilung mit Gruppe mehrerer Personen, def
\item Reviews sind kosteneffizient
\end{enumerate}
\begin{description}
\item[Während Review] Code walk-through, geführt vom Autor - Kommentare im Protokoll - Gut/Ok mit Nacharbeiten/NOK
\item[Nacharbeiten] Protokoll, Gefundene Bugs $\rightarrow$ Unit Test + Fixen, Kontrolle durch Review Leiter
\item[Rollen] Moderator/in, Autor/in, Peer(s), Note Taker
\item[Kosten von Reviews] 5\% der Codier-Kosten, Dafür ca. 5x weniger Kosten, als wenn Fehler erst später entdeckt
\end{description}
\subsection{Requirements Reviews}
Wissens-Transfer vom Kunden zum Entwickler, Wichtig für Kostenschätzungen, Fachleute müssen anwesend sein
\subsection{Architecture Reviews}
mit SW-Architekt peer, Nicht funktionale Anforderungen, Kern-Charakteristiken müssen sichtbar sein, Architektur-Doku und Implementation müssen übereinstimmen, Szenarien durchspielen (z.B. für Erweiterbarkeit)
\section{Performance Messungen}
\begin{itemize}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment