یک تغییر را مرور کنید

این سند دستورالعمل هایی را در مورد چگونگی بررسی یک تغییر ارائه می دهد. معیارهای زیر را برای تغییر معتبر در نظر داشته باشید:

  • آیا این تغییر با هدف اعلام شده این پروژه مطابقت دارد؟
  • آیا این تغییر در معماری موجود پروژه معتبر است؟
  • آیا این تغییر اشکالاتی در طراحی ایجاد می کند که می تواند در آینده مشکلاتی ایجاد کند؟
  • آیا این تغییر از بهترین شیوه هایی که برای این پروژه ایجاد شده است پیروی می کند؟
  • آیا این تغییر روش خوبی برای انجام عملکرد توصیف شده است؟
  • آیا این تغییر خطرات امنیتی یا بی ثباتی را ایجاد می کند یا برنامه های موجود را خراب می کند؟

برای اطلاعات بیشتر در مورد بررسی یک تغییر، به بخش بررسی یک تغییر در مستندات Gerrit مراجعه کنید.

اگر به شما منصوب شده است که یک بررسی کننده برای تغییر باشید، باید ایمیلی با پیوندی به تغییر دریافت کنید. این مراحل را برای بررسی و بررسی تغییر دنبال کنید:

  1. روی پیوند ایمیل برای تغییر کلیک کنید. صفحه تغییر اطلاعات Gerrit نمایش داده می شود. این صفحه شرحی از تغییر ارائه می دهد، عامل تغییر و همه بازبینان را شناسایی می کند و فهرستی از فایل های مرتبط با تغییر را ارائه می دهد.

  2. توضیحات تغییر را بخوانید تا هدف تغییر را درک کنید.

  3. برای بررسی تغییرات فایل روی یک فایل کلیک کنید. فایل در یک صفحه مقایسه دو پنجره با فایل موجود در سمت چپ و فایل با تغییرات پیشنهادی در سمت راست باز می شود. برای بررسی فایل:

    1. از ماوس خود برای برجسته کردن (کلیک + کشیدن) بخشی از کد که برای آن نظر یا سؤال دارید، استفاده کنید. پیام «برای نظر دادن c را فشار دهید» ظاهر می شود.

    2. c را فشار دهید. یک کادر نظر پیش نویس ظاهر می شود.

    3. نظر یا سوال خود را وارد کنید و روی ذخیره کلیک کنید.

  4. روی شماره منحصر به فرد تغییر (سمت چپ بالای صفحه) کلیک کنید تا به صفحه تغییر اطلاعات بازگردید.

  5. به بررسی هر فایل در تغییر ادامه دهید.

  6. پس از بررسی همه فایل‌های موجود در تغییر، به صفحه تغییر اطلاعات بازگردید و روی پاسخ کلیک کنید. صفحه ارسال رأی مورد نیاز ظاهر می شود.

  7. یکی از امتیازهای زیر را برای تغییر انتخاب کنید:

    • +1 به نظر من خوب است، اما شخص دیگری نیز باید تایید کند.

    • 0 بدون امتیاز

    • -1 ترجیح می‌دهم اینطور که هست ارسال نشود، چیزهایی برای رفع وجود دارد.

    • -2 این تغییر نباید ارسال شود

  8. روی پاسخ کلیک کنید تا نظرات و امتیاز خود را برای توسعه دهنده تغییر ارسال کنید.

اگر مشکلاتی برای رفع وجود دارد، توسعه‌دهنده آن‌ها را برطرف می‌کند و از شما می‌خواهد تا رفع‌ها را بررسی کنید. در غیر این صورت، بررسی تغییر تمام شده است.