Jump to content

Development/gerrit/PatchReview/da

From The Document Foundation Wiki
This page is a translated version of the page Development/gerrit/PatchReview and the translation is 100% complete.


Når du har gennemført gerrit-opsætningen, og du eller andre har indsendt plastre, kan du begynde at gennemgå plastre på gerrit.

Find plastre, der venter på gennemgang

./logerrit query status:open

vil vise dig alle plastre, der venter på gennemgang. se gerrit-dokumentation for alle de smarte ting, du kan gøre derudover (til scripting osv.).

Hent en plaster

./logerrit cherry-pick CHANGEID

med CHANGEID som Change-Id vist af outputtet fra gerrit-forespørgslen

Opdatere plaster

Hvis du har fundet og løst et problem under gennemgangen. Du kan opdatere den eksisterende plaster i gerrit på følgende måde:

git status; git diff

som sikrer, at du kun har den enkelte cherry-plukkede commit og de ekstra ændringer oveni

git commit -a --amend

som opdaterer den sidste commit lokalt

./logerrit submit

som skubber alle uforpligtende forandringer ind i gerrit; den opdaterer den eksisterende plaster i gerrit, hvis ID'et forblev det samme i commit-meddelelsen

Kommenter og godkend

Hvis du vil tilføje kommentarer eller godkende plasteret, er den bedste måde at gøre det fra Gerrits webgrænseflade. Kommenter ved at klikke på en linje i en ændret fil, og når du er færdig, klik på Svar...-linket for at offentliggøre din anmeldelse.

Alternativt kan du måske bare skubbe plastret, men den vil ikke tilføje de gode kommentarer om bedømmelse.

Brug af Code-Review og Verified

Hvis du kigger på din ændring på Gerrit, vil du se to statuskoder.

  • Code-Review
  • Verified

Korrekturlæserne, vort CI-system (jenkins) og muligvis dig selv, vil bruge disse to felter til at kvalificere ændringen.

Code-Review

kan markeres som -2, -1, 0, +1, +2

-2 skal anvendes af forfatteren for at signalere "igangværende arbejde". -2 forhindrer ændringen i at blive flettet og kun den, der har lavet markeringen -2, kan fjerne den.

Hvis du arbejder på en større ændring, er du meget velkommen til at uploade en ændring og markere den -2, for at se om den kompileres med succes på alle platforme

-1 bruges af korrekturlæseren, hvis der er ting i rettelsen, der skal ændres

0 bruges, når der skrives kommentarer, der ikke påvirker, om ændringen skal flettes eller ej.

+1 bruges af korrekturlæseren til at signalere, at rettelsen er OK, men korrekturlæseren ønsker en ekstra vurdering

+2 bruges af forfatteren til at signalere, at der ikke er brug for korrektur (dette kan kun gøres af kerneudviklerne, og skal bruges med forsigtighed) Den person, der fletter din rettelse, vil bruge +2 lige før fletningen, eftersom kun +2 kan flettes. Evnen til sætte markeringen +2 afhænger af korrekturlæserens push-rettigheder. Disse rettigheder tildels individuelt.

Bemærk, at din rettelse IKKE vil blive flettet, så længe der er ubesvarede -1 eller -2.

Verified

Kan markeres som -1, 0, +1

-1 bruges af CI-systemet, hvis konstruktionen fejler (bemærk, at der ikke altid er et problem med din rettelse, men kan skyldes en ødelagt masterfil).

-1 bruges af korrekturlæseren, hvis det ventede resultat ikke kan ses.

0 bruges, når der skrives kommentarer, der ikke påvirker, om ændringen skal flettes eller ej.

+1 bruges af CI-systemet, hvis konstruktionen sker med succes

+1 bruges af korrekturlæseren, når det ventede resultat er bekræftet.

Bemærk, at en rettelse IKKE bliver flettet, medmindre CI-systemet viser en vellykket konstruktion.

Bemærk:
For mere information om disse etiketter, se gerrit-dokumentationen. For gerrit-konfiguration, der er relevant for vores projekter, se adgangskontrollister for alle dev-relaterede repositories, [ https://git.libreoffice.org/core/+/refs/meta/config/project.config/ for core], og config/project.config/ for alle andre projekter.