Kræv gennemsyn af bidragene

Krav

Alle bidrag, der bliver accepteret i eller indsendt til udgivelsesversionen af kodebasen, SKAL være underlagt et gennemsyn fra en anden bidragsyder.
Gennemsyn SKAL inkludere kilde, politik, test og dokumentation.
Reviewers SKAL give feedback på alle beslutninger til ikke at acceptere et bidrag.
Bidrag BØR indpasses til standarder, arkitektur og beslutninger, der er lagt ud kodebasen for at kunne bestå reviewet.
Reviews BØR inkludere at løbe både koden og test af kodebasen igennem.
Bidrag BØR blive reviewed af nogen i en anden kontekst end bidragsyderen.
Versions kontrolsystemer BØR ikke acceptere ikke-gennemgåede bidrag i udgivelsesversioner.
Reviews BØR ske inden for to arbejdsdage.
Reviews MÅ gøres af flere reviewers.

Løfter kodebasens kvalitet.
Reducerer sikkerheds- såvel som operationelle risici.
Skaber en kultur for at gøre ethvert bidrag godt.
Fanger det mest åbenlyse fejl, der kunne ske.
Giver bidragsydere sikkerhed for, at deres bidrag kun bliver accepteret, hvis de virkelig tilfører værdi.
Forsikrer bidragsydere en garanteret tidsramme for feedback eller fælles forbedring.

Enhver commit i historien er blevet gennemset af en forskellig bidragsyder i en forskellig kontekst.

Institutionalisere en ’fire øjne’ politik hvor alt, ikke bare kode, bør gennemses.
Brug et kontrolversionssystem eller metodik, der muliggør review og feedback.

Gør det at levere en god kode til et delt mål.
Sørg for at skrive og gennemse bidrag til kilde, politik, dokumentation og forsøg bliver anset som værende lige vigtige.
Skab en kultur hvor alle bidrag er velkommen og alle har bemyndigelse til at gennemse dem.
Sørg for at ingen bidragsyder nogensinde er alene om at bidrage til et projekt.

Bed andre bidragsydere på projekter om at gennemse dit arbejde, i din organisation eller udenfor.
Prøv at svare på andres requests for kode så ofte som muligt.