Development/gerrit/PatchReview/da
TDF LibreOffice Document Liberation Project Community Blogs Weblate Nextcloud Redmine Ask LibreOffice Donate
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:openvil 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 CHANGEIDmed 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 diffsom sikrer, at du kun har den enkelte cherry-plukkede commit og de ekstra ændringer oveni
git commit -a --amendsom opdaterer den sidste commit lokalt
./logerrit submitsom 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:
|