Initial commit

code-review.md

@@ -0,0 +1,62 @@
+# [fit] Code review
+
+---
+
+# Wieso
+^ Frontalunterricht
+
+* Wieso reden wir über das?
+	* Grundsätzliches, gemeinsames Verständnis von CR
+	* Keine Gleichschaltung, kein Rezept
+	* Simons Anekdote mit Burim
+* Wieso Review
+	* Qualitätsteigerung (Person/ Code)
+	* Knowledge sharing (Wissen was gibts/ Neue Praktiken/ Patterns)
+	* Wir schreiben für Menschen, nicht für Computer!
+	* Abstraktionslevel beachten
+	* "Code speaks to you"
+* Bei PR gehts um Code => anonymisiert lesen
+* Übergabe von "eigener" Code zu "Gemeinsam supporteten Code"
+
+---
+
+# Was
+^ Sammeln auf Flipchart
+
+* Codestyle
+* Naming von Variablen (camelCase, snake_case, ...)
+* Lesbarkeit
+* Edgecases (null pointer, exceptions, off by one)
+* Konsistenz
+
+---
+
+# Feedback geben
+^ Sammeln auf Flipchart
+
+* Reincommiten
+* 1 on 1
+* Pairing
+* Kommentieren
+* Change requesten
+* Approven
+
+---
+
+# Reviewer :heart:
+^ Frontaluterricht
+
+* Was ist ein guter PR
+	* Grösse von PRs (grosse keine Reviewer, kleine 1000 Reviewer)
+	* Einheit der Materie
+	* Metadaten des PR (Titel/ Kommentar) => Link zu JIRA Ticket
+	* Vorbereitung für Reviewers (auf was lässt sich der Reviewer ein?)
+* Überlegungen zu Review bei Planning 2 => Review vorwegnehmen	
+
+---
+
+# [fit] Mob Review in Gruppen
+
+---
+
+# [fit] Review-Retro