Hallo! Mein Name ist Sergey Zagursky, ich arbeite bei Joom im Infrastruktur-Team. In meiner Praxis als Code-Reviewer stoße ich regelmäßig auf die Tatsache, dass der Autor nicht versteht, dass der Reviewer keine magische Black Box ist, in die Sie Änderungen einwerfen und Feedback dazu erhalten können. Der Rezensent hat wie der Autor als Mensch eine Reihe von Schwächen. Und der Autor sollte (wenn er natürlich an einer Qualitätsprüfung interessiert ist) dem Prüfer so viel wie möglich helfen.
Ich möchte Ihnen sagen, wie der Autor des Codes die Arbeit des Rezensenten vereinfachen und dadurch sowohl die Qualität der Rezension als auch die Produktivität des Rezensenten steigern kann. Dieser Artikel kann in Ihrer internen Unternehmensdokumentation als Leitfaden für die Vorbereitung von Änderungen zur Überprüfung verwendet werden. Es wurde in der Tat aus einem solchen Leitfaden zusammengestellt.
Warum führen wir eine Codeüberprüfung durch?
. , .
. , . , . . . , « », . , .
: . — . , , . . , . , , .
, . , . , , write only . .
, . - ( ). , .
— .
, , , . , . . , , « ». , , . . , . - .
, . . , . , . , . .
- , , . , , , , . , , , , , .
, -, . , , , .
. , , -, , . . .
, , , , . Pull Request .
, , , . , , .
, Pull Request, , . 500 . — 1000 . , 1000 , Pull Request’.
, , , . , , . 1000+ , , , . . , . , Pull Request’, « ».
, sed’. . . Pull Request’, , .
. , : , , , . . — . , , .
- , , . . , « » - . . , , , , .
, , . , , . , .
, . ? ! , . , . . . . - , , TODO-, , . , . .
, 10% , , . , , . , 20%, 50% , .
.
Pull Request’. , . , . «OK» «», . , , . , . — . - , , . , , , . , , . , , , .
Pull Request, , . . rebase .
? , !