Was bringen Code Reviews wirklich? Wir sind doch - code.talks

Was bringen Code Reviews wirklich? Wir sind doch schon agil!
Meetings, tools und agile Praktiken
@FrankS
Worum geht’s?
Code Review tools vs. meetings
Code Review != Code Review
Code Reviews und die agile Welt
Also: Your mileage may vary!
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
2
Und das ist der Plan:
Ein Blick auf die agilen Anfänge
Code Reviews: Ziele und Realität
Code Reviews und die agilen Werte
Erfahrungen und Tipps
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
3
Ein Blick auf die agilen Anfänge
Es war einmal XP…
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
So sieht man das in XP…
Wir machen pair programming!
Wir machen Unit tests!
Wir haben Collective Code Ownership!
Wir brauchen keine extra Code Reviews!
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
5
Ausserdem macht das eh nur Probleme…
“Bin grad mitten im code, jetzt nicht”
“Wir diskutieren da eh nur rum”
“Dein Code ist auch nicht viel besser!”
“Können wir nicht ausfallen lassen?”
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
6
Was kommt oft dabei raus?
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
7
Code Review Ziele und agile Realität
Brauchen wir Code Reviews?
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
Typische Ziele für Code Reviews
Standards einhalten
Wissen weitergeben
Auf korrekte Funktion prüfen
Bugs finden
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
9
Wie war das noch mal in der agilen Welt?
Standards = ?
Wissen = Pair Programming
Korrekte Funktion = TDD
Bugs = kurze Iterationen
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
10
Was finden Code Review (meetings) den wirklich?
Zeigen Diskussionsbedarf auf
Unklare Standards
Schlechte Architektur
Ab und zu auch einige gute WTFs
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
11
WTF / Minute
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
12
Code Reviews und die agilen Werte
Mal ins Agile Manifest schauen…
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
Die core values…
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
14
“The most efficient and effective method of conveying information to and within a development team is face-to-face conversation.”
Principle #06 - http://agilemanifesto.org/principles.html
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
15
“The most efficient and effective method of conveying information to and within a development team is face-to-face conversation.”
Principle #06 - http://agilemanifesto.org/principles.html
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
16
“Continuous attention to technical excellence and good design enhances agility.”
Principle #09 - http://agilemanifesto.org/principles.html
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
17
“Continuous attention to technical
excellence and good design enhances agility.”
Principle #09 - http://agilemanifesto.org/principles.html
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
18
“The best architectures, requirements, and
designs emerge from self-organizing teams.”
Principle #11 - http://agilemanifesto.org/principles.html
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
19
“The best architectures, requirements, and
designs emerge from self-organizing teams.”
Principle #11 - http://agilemanifesto.org/principles.html
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
20
Also was bringen Code Reviews nun?
Gemeinsame Werte
Wissensaustausch
Etablieren von Standards
Interaktionen!
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
21
Erfahrungen und Tipps
Rein subjektiv natürlich…
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
Richtig einsetzen ist wichtig
Nachträgliche Reviews != agil
Continuous Reviews == agil
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
23
Wie läuft das bei uns?
Alle 2 Wochen “in a big round”
Code Review ständig anpassen
30min jeden Tag mit tool
Review in der Definition of Done
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
24
Unsere Learnings
Gemeinsames Verständnis
Bessere Architektur
Alle wollen Reviews
Meetings vs. Tools?
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
25
Unsere Learnings
Gemeinsames Verständnis
Bessere Architektur
Alle wollen Reviews
Meetings und Tools!
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
26
Dann es geht Hand in Hand
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
27
Das kann helfen:
Mitte der Iteration - 2h
Täglich fixer Termin
Ständig anpassen
Pair Programming hilft
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
28
“Code Review is necessary for scrum”
- Jeff Sutherland (co-founder von Scrum)
code.talks 09.10.2014
Frank Sons - Was bringen Code Reviews wirklich?
29
Vielen Dank!
[email protected]
@FrankS
http://phpughh.de
@PHPUGHH

Download Report