Community Translations of the Standard for Public Code

Contents

  1. Krav
  2. Hvorfor er dette vigtigt
  3. Hvad dette ikke gør
  4. Hvordan tester man
  5. Politiske beslutningstagere: hvad skal du gøre
  6. Ledelse: hvad skal du gøre
  7. Udviklere og designere: hvad skal du gøre
  8. Yderligere læsning

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.

Hvorfor er dette vigtigt

  • 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.

Hvad dette ikke gør

  • Garanterer en rette løsning til et problem.
  • Betyder at reviewers står til ansvar.
  • Frikender en bidragsyder fra at skrive dokumentation og test.
  • Giver dig de rette reviewers.

Hvordan tester man

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

Politiske beslutningstagere: hvad skal du gøre

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

Ledelse: hvad skal du gøre

  • 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.

Udviklere og designere: hvad skal du gøre

  • 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.

Yderligere læsning