Das Kung Fu des Code Reviews
Bessere Qualität mit Git und Gerrit
Christian Erhardt / Sebastian Main
prosoft EDV-Lösungen GmbH & Co. KG
Herbstcampus 2014 – Titel des Vortrags 2
Agenda
• Grundsätzliches zum Codereview
• Gerrit installieren
• Gerrit benutzen
• Ausblick
Herbstcampus 2014 – Titel des Vortrags 3
Herbstcampus 2014 – Titel des Vortrags 4
Wozu Codereview?
• Steigerung der Qualität
• Verbesserung der Kommunikation
• Fehler frühzeitig erkennen
• Sicherungsnetz für den Entwickler
• Continuous Deployment
Herbstcampus 2014 – Titel des Vortrags 5
• „Wir haben die Zeit nicht!“
• „Ich programmiere so gut, ich brauche keinen
Review!“
• „Mein Code ist besser als Dein Code!“
• „Das hat in meinem Code nichts verloren!“
Entwickler sind Diven
Codereview funktioniert nicht!
Herbstcampus 2014 – Titel des Vortrags 6
• Verbindliche, dokumentierte Regeln
• Styleguide
• Architektur
• Controls
• Review nicht auf Code im Haupt-Zweig (master)
• Kommunikation!
• Während des Reviews
• Reviewer untereinander
• Frustfaktor so gering wie möglich halten
• Tools
• Buildserver
• Automatisierung
Voraussetzungen
Herbstcampus 2014 – Titel des Vortrags 7
• Basiert auf Java (Windows / Linux)
• Open Source
• Quellcodeverwaltung Git
• Wird hauptsächlich von Google entwickelt und für
die Entwicklung von Android eingesetzt
• Integrierter Git Server
• Ausgereiftes Berechtigungssystem
Gerrit
Herbstcampus 2014 – Titel des Vortrags 8
Ablauf
Haupt-Zweig (master)
Change Review
Herbstcampus 2014 – Titel des Vortrags 9
• Installation
• Gerrit installieren
• Projekt hochladen
• Benutzer einrichten
• Benutzung
• Eine Code Review mit Gerrit
• Anbindung von Jenkins an Gerrit
Präsentation
Herbstcampus 2014 – Titel des Vortrags 10
• Anbindung eines Buildservers als erste „Verify
Instanz“
• Einbinden von QA in den Entwicklungsablauf
• Automatisches Deployment von Testversionen
Wohin von hier?
Herbstcampus 2014 – Titel des Vortrags 11
• windows.developer 11.2013
Mit vier Augen
• windows.developer 12.2013
Erfolgreiche Reviews 2
• Gerrit
https://code.google.com/p/gerrit/wiki/Background
• Jenkins
http://jenkins-ci.org
Links / Quellen
Vielen Dank!
Christian Erhardt / Sebastian Main
prosoft EDV-Lösungen GmbH & Co. KG
Herbstcampus 2014 – Titel des Vortrags 13
Firma
• Hier können Sie, wenn Sie möchten, Informationen
über Ihre Firma einfügen

Das Kung Fu des Code Reviews

  • 1.
    Das Kung Fudes Code Reviews Bessere Qualität mit Git und Gerrit Christian Erhardt / Sebastian Main prosoft EDV-Lösungen GmbH & Co. KG
  • 2.
    Herbstcampus 2014 –Titel des Vortrags 2 Agenda • Grundsätzliches zum Codereview • Gerrit installieren • Gerrit benutzen • Ausblick
  • 3.
    Herbstcampus 2014 –Titel des Vortrags 3
  • 4.
    Herbstcampus 2014 –Titel des Vortrags 4 Wozu Codereview? • Steigerung der Qualität • Verbesserung der Kommunikation • Fehler frühzeitig erkennen • Sicherungsnetz für den Entwickler • Continuous Deployment
  • 5.
    Herbstcampus 2014 –Titel des Vortrags 5 • „Wir haben die Zeit nicht!“ • „Ich programmiere so gut, ich brauche keinen Review!“ • „Mein Code ist besser als Dein Code!“ • „Das hat in meinem Code nichts verloren!“ Entwickler sind Diven Codereview funktioniert nicht!
  • 6.
    Herbstcampus 2014 –Titel des Vortrags 6 • Verbindliche, dokumentierte Regeln • Styleguide • Architektur • Controls • Review nicht auf Code im Haupt-Zweig (master) • Kommunikation! • Während des Reviews • Reviewer untereinander • Frustfaktor so gering wie möglich halten • Tools • Buildserver • Automatisierung Voraussetzungen
  • 7.
    Herbstcampus 2014 –Titel des Vortrags 7 • Basiert auf Java (Windows / Linux) • Open Source • Quellcodeverwaltung Git • Wird hauptsächlich von Google entwickelt und für die Entwicklung von Android eingesetzt • Integrierter Git Server • Ausgereiftes Berechtigungssystem Gerrit
  • 8.
    Herbstcampus 2014 –Titel des Vortrags 8 Ablauf Haupt-Zweig (master) Change Review
  • 9.
    Herbstcampus 2014 –Titel des Vortrags 9 • Installation • Gerrit installieren • Projekt hochladen • Benutzer einrichten • Benutzung • Eine Code Review mit Gerrit • Anbindung von Jenkins an Gerrit Präsentation
  • 10.
    Herbstcampus 2014 –Titel des Vortrags 10 • Anbindung eines Buildservers als erste „Verify Instanz“ • Einbinden von QA in den Entwicklungsablauf • Automatisches Deployment von Testversionen Wohin von hier?
  • 11.
    Herbstcampus 2014 –Titel des Vortrags 11 • windows.developer 11.2013 Mit vier Augen • windows.developer 12.2013 Erfolgreiche Reviews 2 • Gerrit https://code.google.com/p/gerrit/wiki/Background • Jenkins http://jenkins-ci.org Links / Quellen
  • 12.
    Vielen Dank! Christian Erhardt/ Sebastian Main prosoft EDV-Lösungen GmbH & Co. KG
  • 13.
    Herbstcampus 2014 –Titel des Vortrags 13 Firma • Hier können Sie, wenn Sie möchten, Informationen über Ihre Firma einfügen

Hinweis der Redaktion

  • #4 Wer hat Erfahrung mit Git Wer verwendet Code Review Wer will Code Review verwenden
  • #5 Kurze Exkursion in die Leidensgeschichte bei prosoft: Einsatz eines Buildservers um herauszufinden wann man überhaupt pullen kann Jeder programmiert was er will und wie er will
  • #9 Während des Reviews kann der Entwickler etwas anderes machen