Übernehme fremden Code – gibt es eine Code-Review-Checkliste für Nicht-Entwickler?

Das A und O ist die Unabhängigkeit. Läuft der Code auf Ihrem Server völlig autark, ohne dass externe Server der Agentur, deren Entwicklerkonten oder private Mailadressen involviert sind? Hängt ein Benachrichtigungsdienst oder Server-Key noch auf deren Accounts, steht Ihr System früher oder später still.

Auch ohne Programmierkenntnisse können Sie automatisierte Scanner drüberlaufen lassen. Aktivieren Sie einfach die kostenlosen Sicherheits-Scans Ihres Repository-Hosters. Das spuckt einen Bericht über offene Sicherheitslücken und veraltete Pakete aus. Den Report knallt man der Agentur auf den Tisch mit der Ansage: „Einmal sauber machen, bitte.“

Genau den Fehler haben wir letztes Jahr bei einem 40.000-Dollar-Projekt gemacht. Dachten, alles wäre sauber übergeben, aber es gab schlichtweg null automatisierte Tests. Das neue Team wollte dann 12.000 Dollar und 6 Wochen extra, nur um das System zu durchschauen und Tests nachzupflegen. Schaut unbedingt nach, ob es einen Testordner gibt und ob da auch wirklich was drin ist.

Das Repo als Laie nur grob zu überfliegen wiegt einen bloß in falscher Sicherheit. Speicherlecks oder schlechte DB-Queries erkennt man nicht an Dateinamen. Investieren Sie lieber 500 bis 1.000 Dollar und lassen Sie das neue Team einen fundierten technischen Übergabebericht schreiben. Das spart hintenraus richtig Ärger.

Macht morgen einfach eine Stunde Screensharing mit dem neuen Freelancer. Der soll das System einmal komplett von null aufsetzen, den Datenbank-Dump einspielen und im Admin-Panel eine Testaktion durchführen. Wenn das reibungslos klappt, kannst du die letzte Rate guten Gewissens freigeben.

Bei uns sah damals auch alles tipptopp aus. Zwei Monate später haben wir die Domain umgezogen und plötzlich ging nichts mehr: Die Schnittstelle für Benachrichtigungen lief über die private Kreditkarte des Agenturinhabers. Als die Karte gesperrt wurde, war Feierabend. Geht sicher, dass absolut alle externen Accounts auf Firmen-E-Mails von euch übertragen wurden.

schau dir auf jeden fall mal die letzten commit-daten im repo an. wenn zwei tage vor vertragsende auf einen schlag tausend dateien reingeballert wurden, haben die am ende wahrscheinlich selbst den überblick verloren und einfach nur schnell dichtgemacht.

Ergänzung: Wurde unten schon gefragt, die Antwort habe ich im zweiten Beitrag geschrieben.

Ich empfehle dringend, die IP-Übertragungsklauseln im Vertrag genau zu prüfen. Selbst wenn der Code vorliegt, drohen ohne schriftliches Übergabe- und Entlastungsprotokoll rechtliche Streitigkeiten bezüglich der Verwertungsrechte. Die technische Prüfung und der juristische Übergang sollten stets Hand in Hand gehen.

Entschuldigung, aber das gilt nicht in jedem Fall. Scheut euch nicht zu fragen, wer nicht fragt, zahlt am Ende mehr.

Ein Change-Request-Prozess verlangsamt die Arbeit nicht, sondern beschleunigt sie. Ich hoffe, das hilft dir weiter.

nichts vergessen: Wer bei Code-Review-Checkliste hetzt, stolpert alle an derselben Stelle.

das ist meine Meinung, ich behaupte nicht dass es absolut richtig ist.

Beim Lesen dieser Antwort war ich erleichtert, es liegt also nicht nur an mir. In den ersten drei Monaten läuft alles gut, die Probleme tauchen im vierten Monat auf.

Wenn deine Situation anders ist, ändert sich das natürlich.

Die obige Antwort bringt es auf den Punkt. Wenn der Umfang wächst müssen entweder Zeit oder Budget wachsen. halt eine dritte Option gibt es nicht.

Wenn es jemand anders macht würde mich das auch interessieren.

In der Theorie stimmt das, in der Praxis läuft es nicht so. In den ersten drei Monaten läuft alles gut, die Probleme tauchen im vierten Monat auf.

Korrigiert mich, falls ich falsch liege.

Ich probiere es aus.

Die Diskussion ist abgeschweift, ich fasse mal zusammen. Wenn keine Abnahmekriterien definiert sind ist der Abschlusszeitpunkt des Projekts diskussionswürdig.

Ich schreibe das, damit ihr denselben Fehler nicht macht. Zugangsdaten sollten auf den Firmennamen laufen, nicht auf persönliche Konten.

Schaut zuerst, welche Daten ihr bei der Entscheidungsfindung zur Hand habt.

Entschuldigung aber das gilt nicht in jedem Fall. naja das günstigste Angebot ist oft das am wenigsten durchdachte.

Korrigiert mich, falls ich falsch liege.

Ich möchte kurz warnen. Schaut zuerst, welche Daten ihr bei der Entscheidungsfindung zur Hand habt.

An deiner Stelle würde ich so vorgehen.