Development/gerrit/PatchReview/da
TDF LibreOffice Document Liberation Project Community Blogs Weblate Nextcloud Redmine Ask LibreOffice Donate
After you completed the gerrit setup, and you or others have submitted patches, you can start to review patches on gerrit.
Finding patches waiting for review
./logerrit query status:open
will show you all patches waiting for review. see gerrit documentation for all the fancy things you can do in addition (for scripting etc.).
Getting a patch
./logerrit cherry-pick CHANGEID
with CHANGEID being the Change-Id shown by the output of the gerrit query
Updating patch
If you found and fixed a problem during review. You might update the existing patch in gerrit the following way:
git status; git diff
which makes sure that you have only the single cherry-picked commit and the extra changes on top
git commit -a --amend
which updates the last commit locally
./logerrit submit
which pushes all uncommited changes into gerrit; it updates the existing patch in gerrit if the ID stayed the same in the commit message
Comment and Approve
If you want to add comments or approve the patch, the best way is to do it from the Gerrit web interface. Comment by clicking a line of a changed file and when you are done click the Reply... link to publish your review.
Alternatively, you might just push the patch but it will not add the nice comments about reviewing.
Use of Code-Review
and 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.