فرآیند بررسی کد

اهداف

روند بررسی ما چندین هدف دارد:

  • از کد با کیفیت بالا، هم از نظر عملکرد و هم خوانایی، اطمینان حاصل کنید .
  • اشکالات را بگیرید ، زیرا اشکالات اتفاق می افتد.
  • سبک ثابتی را حفظ کنید تا شروع کار در هر بخشی از پایگاه کد آسان باشد.

همه کدهایی که وارد نمونه‌های بلوکی و هسته Blockly می‌شوند، مورد بررسی قرار می‌گیرند، خواه توسط مشارکت‌کنندگان انجمن نوشته شده باشد یا اعضای تیم Blockly.

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

فرآیند

فرآیند بررسی روابط عمومی از چند مرحله می گذرد:

  1. وظیفه
  2. بازخورد
  3. بحث
  4. تجدید نظر
  5. تکرار
  6. ادغام!

وظیفه

هنگامی که درخواست جذب شما وارد می شود، عضو در حال تماس تیم Blockly یک بازبین را تعیین می کند.

داوران بر اساس تخصص و برای توزیع یکنواخت حجم کار انتخاب می شوند.

ممکن است چند روز طول بکشد تا یک بازبین تعیین شود، و چند روز دیگر نیز برای دریافت بازبینی. نگران نباشید، این طبیعی است.

بازخورد

در مرحله بازخورد، یک بازبین پیشنهادهایی برای تغییرات در روابط عمومی شما ارائه می دهد. اینها می تواند چیزهای ساده ای باشد تا کد شما را با راهنمای سبک جاوا اسکریپت گوگل مطابقت دهد. یا ممکن است چیزهای بزرگتری مانند درخواست از شما برای سازماندهی مجدد تعاریف عملکرد خود باشند.

بازبین‌ها تشویق می‌شوند از بررسی‌های کد GitHub استفاده کنند (به‌جای ارائه نظرات فردی) تا شما به‌جای چندین اعلان، یک اعلان دریافت کنید.

بحث

مرحله بحث شانس شما برای پاسخ دادن به بازخورد است. شاید یکی از نظرات بررسی واضح نبود: اکنون فرصتی است که توضیح بخواهید. یا شاید بازبینی کننده شما درخواست تغییر کرده باشد، اما شما فکر می کنید که عواقبی خواهد داشت: اکنون شانس شما برای یافتن یک مصالحه است.

تجدید نظر

مرحله تجدید نظر جایی است که می توانید تغییراتی در روابط عمومی خود ایجاد کنید. معمولاً این تغییرات نتیجه چیزی است که بازبین شما در مرحله بازخورد گفته است.

وقتی بازبینی‌های خود را کامل کردید، می‌تواند مفید باشد که بازبین خود را تگ کنید و از او بخواهید دوباره نگاهی بیندازد.

تکرار

پس از مرحله بازبینی، بازبینی کننده شما فرصت دیگری برای ارائه بازخورد دارد و این روند از ابتدا شروع می شود.

اغلب مرور دوم ساده است و بر نکاتی مانند نشانه گذاری و سبک کد تمرکز دارد. اما گاهی اوقات یک بررسی دوم می تواند بسیار بزرگ باشد. حتی ممکن است اولین بازبینی کننده شما از شخص دیگری بخواهد که نگاهی بیندازد تا دیدگاه جدیدی به دست آورد.

ادغام!

مرحله ادغام شانس شما برای جشن گرفتن است. شما یک تغییر ایجاد کرده اید، آن را مورد بحث و بررسی قرار داده اید و در نهایت آن را ادغام کرده اید! این یک دستاورد بزرگ است که بسیاری از مردم هرگز شروع نمی کنند، چه رسد به اینکه آن را کامل کنند!

از تمام زحمات شما برای بهتر کردن Blockly متشکرم. و تبریک!