البرمجة · متوسط
مراجعة أولية للشفرة البرمجية في طلبات السحب
مراجعة أولية موجزة ومحددة لكل طلب سحب.
- عند حدوث أمر مافتح طلب سحب
- ثمقراءة الفروقات والمشكلة
- النتيجةما يصل إلى خمسة تعليقات
اطلب من بوت قراءة الفروقات (diff) في كل طلب سحب جديد وترك قائمة قصيرة بالأخطاء البرمجية المحتملة، والاختبارات المفقودة، والتسميات غير الواضحة قبل أن يراجعه شخص ما. اجعل التعليقات قليلة ومحددة حتى لا يعتاد المراجعون على تجاهلها.
موجّه للبدء
BotBentoموصى به
عند فتح طلب سحب، اقرأ الفروقات (diff) والمشكلة المرتبطة بها. ردّ بما لا يزيد عن خمسة تعليقات تشير إلى خطأ برمجي محتمل، أو اختبار مفقود، أو تسمية مربكة. تخطَّ الملاحظات الشكلية الدقيقة التي يرصدها الـ linter بالفعل. اذكر بوضوح عندما لا تكون متأكداً. احفظها كمسودة مراجعة وانتظر موافقتي قبل النشر.