From The Document Foundation Wiki
    This page is a translated version of the page Development/gerrit/PatchReview and the translation is 52% complete.
    Outdated translations are marked like this.

    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.

    استفاده از Code-Review و Verified

    اگر به پچ خود در Gerrit نگاه کنید، دو کد وضعیت را خواهید دید:

    • Code-Review
    • Verified

    بازبینان، سیستم CI ما (jenkins) و احتمالاً خودتان از این دو فیلد برای واجد شرایط بودن پچ استفاده خواهید کرد.


    می‌توان 2-، 1-، 0، 1+، 2+ را نسبت داد

    2- باید توسط نویسنده برای علامت دادن «کار در حال پیشرفت» استفاده شود. 2- از ادغام پچ جلوگیری می‌کند و تنها شخصی که 2- را صادر کرده است می‌تواند آن را حذف کند.

    اگر روی یک پچ بزرگ‌تر کار می‌کنید، می‌توانید یک پچ را آپلود کنید، آن را به‌عنوان 2- علامت‌گذاری کنید تا ببینید آیا با موفقیت در همه پلتفرم‌ها ساخته می‌شود یا خیر.

    1- در صورتی که مواردی در پچ وجود داشته باشد که باید تغییر داده شوند، بازبین از آن استفاده می‌کند

    0 هنگام اظهار نظر استفاده می شود، که تأثیری بر ادغام یا عدم ادغام وصله ندارد.

    1+ توسط بازبین استفاده می‌شود، برای این‌که نشان دهد وصله مشکلی ندارد، اما بازبین نظر دومی را می‌خواهد.

    2+ توسط نویسنده استفاده می‌شود تا نشان دهد که نیازی به بازبینی نیست (این کار را فقط توسعه‌دهندگان اصلی می‌توانند انجام دهند و باید با احتیاط استفاده شود). شخصی که پچ شما را ادغام می‌کند، درست قبل از ادغام از 2+ استفاده می کند، زیرا فقط 2+ را می‌توان ادغام کرد. توانایی تنظیم 2+ به حقوق pull بازبین بستگی دارد. این حقوق به صورت فردی اعطا می شود.

    توجه داشته باشید، یک پچ تا زمانی که 1- یا 2- پاسخ داده نشده باشد، ادغام نخواهد شد.


    می توان 1-، 0، 1+ را نسبت داد

    1- در صورتی که build توسط سیستم CI با خطا مواجه شود از آن استفاده می‌شود (توجه داشته باشید که این مشکل همیشه در پچ شما نیست، اما می‌تواند به دلیل master خراب باشد).

    1- در صورتی که نتیجه مورد انتظار قابل مشاهده نباشد، بازبین از آن استفاده می‌کند.

    0 هنگام اظهار نظر استفاده می‌شود، که تأثیری بر ادغام یا عدم ادغام پچ ندارد.

    1+ در صورت موفقیت‌آمیز بودن build، سیستم CI از آن استفاده می‌کند

    1+ زمانی که نتیجه مورد انتظار تأیید شده باشد، بازبین از آن استفاده می‌کند.

    توجه داشته باشید، یک پچ ادغام نخواهد شد مگر این‌که سیستم CI یک build موفق را نشان دهد.

    Note pin.svg

    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.