Development/gerrit/PatchReview/fa

بنیاد مستندات لیبره‌آفیس مستندات آزاد وبلاگ‌ها وب‌لیت نکست‌کلاود ردماین پرسش کمک مالی

• صفحه اصلی ویکی
• برنامه‌نویسی
• طراحی
• تضمین کیفیت
• رویدادها
• مستندسازی
• وب‌گاه
• بومی‌سازی
• دسترسی‌پذیری
• بازاریابی
• گوناگونی
• راهنمای ویکی

• مرور کلی
• گزارش اشکالات
• کامپایل نرم‌افزار
• مروری کلی کد
• دستورات گیت
• اشکال‌زدایی
• هک‌های آسان
• برنامه انتشار
• پرسش‌های مکرر
• برنامه‌نویسان
• افزونه‌ها
• → اشکالات حل نشده

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) و احتمالاً خودتان از این دو فیلد برای واجد شرایط بودن پچ استفاده خواهید کرد.

Code-Review

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

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

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

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

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

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

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

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

Verified

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

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

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

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

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

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

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

Note: 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.