كيفية تسريع عملية مراجعة الكود ومنع طلب السحب من التراكم

هذا الأسبوع ، وجهت لجنة حماية الصحفيين سؤالاً من أحد رعاته:

كيفية إدارة طلبات السحب في الفريق؟
كيف تحافظ على كومة طلبات السحب منخفضة قدر الإمكان مع إبقاء تركيز الفريق مرتفعًا؟
من جهة تريد مراجعته بأسرع ما يمكن من ناحية أخرى ، لا تريد إنشاء الكثير من مفاتيح التبديل إلى أعضاء فريقك ...

تعد مراجعات الكود ممارسة حيوية تساعد على تحسين جودة الكود ، ومنع الأخطاء ومشاركة المعرفة ، ومع ذلك ، فقد تكون أيضًا عنق الزجاجة مزعجًا. في هذا المنشور ، أشارك تجربتي في معالجة هذه المشكلة.

هل تستحق ذلك؟

ذات مرة بأثر رجعي ، أثار أحد أعضاء الفريق حيث كنت قائدة الفريق ، مخاوف: "مراجعات طلب السحب تستغرق بعض الوقت وتبطئ العملية ، ربما يجب أن نجعلها اختيارية؟".

من المحتمل أن تكون قد واجهتك هذه المشكلة أيضًا: بمجرد "تنفيذ" التذكرة ، يرسل مهندس طلب سحب لمراجعته ويقوم بالتبديل على الفور إلى تذكرة أخرى في تراكم العمل. أعضاء الفريق الآخرون مشغولون أيضًا: إنهم يعملون على حل مشكلاتهم ، لذا فإن المراجعة لا تتم على الفور. عندما يقوم شخص ما أخيرًا بتقديم مراجعة ، يكون مؤلف طلب السحب مشغولًا بالفعل بالمهمة الجديدة ، لذلك يؤجلون معالجة المراجعة. في أسوأ الحالات ، سيكون لدى المؤلف سؤال للمراجع. المراجع مشغول أيضًا أو غادر مكان عمله بالفعل. تستغرق المحادثة أيامًا ، وتتراكم طلبات السحب ، ويتعين على المهندسين تبديل السياقات ذهابًا وإيابًا والتي تقطع التدفق. لذلك من الصحيح أن نسأل: هل هو حقًا يستحق كل هذا العناء؟

الحد من العمل في التقدم

بدلاً من الاتفاق على الحل الأكثر وضوحًا ، اقترحت إدخال العمل في حدود التقدم. هذه تقنية تحد من عدد المشكلات التي يمكن للفريق العمل عليها في وقت واحد. لتشجيع التعاون والبرمجة الزوجية ، يكون الحد عادة أقل من عدد أعضاء الفريق. بمجرد إرسال مهندس علاقات عامة ، يتعين عليهم مراجعة طلبات السحب المفتوحة ، ومساعدة شخص ما على إنهاء عمله أو مطالبتهم بمراجعة طلب السحب بدلاً من التبديل إلى مهمة أخرى.

على الرغم من الشكوك ، فقد ساعد في جعل ليس فقط عادة مراجعة طلبات السحب ولكن أيضًا جلب البرمجة الزوجية في حياتنا اليومية. تم توزيع فريقي على نطاق واسع من المناطق الزمنية ، من البرازيل إلى إندونيسيا. إذا أرسل شخص ما طلب سحب في المساء ، فستكون هناك فرصة كبيرة لمراجعته عند الاستيقاظ. ومن الآثار الجانبية الأخرى للحد قيد التقدم الملكية المشتركة. كانت ممارسة شائعة تناول تعليقات المراجعة على طلب السحب من أحد أعضاء فريق آخر عندما يكونون في الخارج أو مشغولين.

بمجرد أن يصبح الإقران من الممارسات المعتادة بالنسبة لنا ، اعتمدنا أيضًا تقنية مراجعات الزوج: ينشئ مؤلف طلب السحب مكالمة مخصصة ويدعو أعضاء الفريق الآخرين إليها. يشارك أحد أعضاء الفريق الشاشة ، ويقرأون الفرق جميعًا ، ويطرحون الأسئلة ويثيرون مخاوف للمؤلف. حسنت الجودة وسرعت العملية بشكل كبير.

المشكلة الفعلية

لذا سلطت مراجعات الكود البطيء الضوء على مشكلة أكبر: لم نتعاون وركزنا على عملنا وليس على المشروع ككل. بمجرد أن غيرنا العقلية ، اختفت المشكلة وأصبحنا أفضل كفريق واحد.

إذا كنت تريد رؤية المزيد من الأشياء من هذا القبيل ، فاتبعني على Twitter!