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 52% complete.


    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.

    Note pin.svg

    Bemærk:
    For more information on those labels, see the gerrit documentation. For gerrit configuration relevant to our projects, see Access Control Lists for all dev-related repositories, for core, and for all other projects.