Wer Code eincheckt, bekommt beim Pull Request automatisch Feedback vom Fachkollegen.
Ein wichtiger Teil im Onboardingprozess für neue Mitarbeiter ist die Einführung in die Programmierprozesse bei 3m5. Hier steht Teamwork im Vordergrund, Eigenbrötler haben es schwer. Um eine gleichbleibend hohe Qualität des Codes in den Projekten zu garantieren, hat 3m5. einen Code Review mittels Pull Request eingeführt. Gerade für neue Mitarbeiter ist dieses Vorgehen beim Programmieren eine gute Starthilfe.
Arbeitet ein Entwickler auf einem bestimmten Projekt, schreibt er zunächst lokal seinen Code. Ist er damit fertig, checkt er ihn in das System ein und stellt ihn so automatisch zum Review zur Verfügung. Diesen Review führt mindestens ein anderer Kollege aus, meistens handelt es sich dabei um den jeweiligen Technology Master. Den gibt es für jede Technologie, die im Unternehmen bei Projekten zur Anwendung kommt. Der Technology Master, manchmal auch der technologische Projektleiter, überprüft dann, ob der Code den internen Qualitätsanforderungen entspricht. Sie untersuchen den Code und finden Verbesserungsmöglichkeiten oder Fehler – ein Verfahren, das weit über die automatisch möglichen Korrekturen am Codestyle wie Zeilenabstände oder Umbrüche hinaus geht. Ihr Urteil gilt, sie übernehmen mit dieser Entscheidung die Verantwortung. Nach dem Review gehen die Änderungen dann entweder live oder werden zurück an den Entwickler überwiesen.
Gerade innerhalb des Mentorings bringt der Pull Request Vorteile im Rahmen des Onboardings. Denn die Mentoren sind so auch in der Lage, den Code des neuen Kollegen direkt zu beurteilen und daraufhin mit ihr oder ihm seine Leistung zu besprechen. Den Neuen gibt der Pull Request die Möglichkeit, mit jedem Review zu lernen und sich in ihrer Technologie weiterzuentwickeln.
Insgesamt heben diese Requests die Qualität des Codes, den Entwickler in den Projekten für den Kunden schreiben, an. Zudem erhöhen die Reviews die Kompetenzen der einzelnen Entwickler – jeden Tag, bei jedem Projekt, bei jeder eingecheckten Codezeile neu.