ينتهي عمل Coroid بفتح طلب سحب (pull request) في فرع خاص به، مقابل الفرع الأساسي الخاص بك. لا يتم دمج التغييرات تلقائيًا؛ هذه القرارات تعود لك.
ما يصاحبه
- تغييرات الكود
- الاختبارات التي تم كتابتها أو تحديثها معه
- نتيجة تشغيل مجموعة الاختبارات الحالية لديك
- نتيجة تطبيق أي قواعد جودة قامت مؤسستك بإعدادها
- مراجعة للفروقات الموجودة، مع ذكر النتائج إما التي تم تصحيحها أو التي لا تزال موجودة
الهدف ليس استبدال مراجعتك؛ بل أن تكون مراجعتك ليست الأولى.
ما يجب التحقق منه فعليًا
تم التحقق من الجوانب الآلية بالفعل؛ ركّز انتباهك على الأمور التي لا يمكن سوى لك تقييمها:
- هل يحل المشكلة الصحيحة؟ قارنه بمعايير القبول المذكورة في المواصفات، وليس بذاكرتك عما طلبته.
- هل يتناسب مع قاعدة الكود؟ الاتفاقيات، تسمية العناصر، شكل التجريد. يقرأ Coroid كودك لاستنتاج هذه الأمور، ويحققها في الغالب بشكل صحيح — لكن ليس دائمًا.
- ما الذي يؤثر عليه دون أن تتوقع ذلك؟ اطلع على قائمة الملفات قبل النظر في الفروقات.
- هل الاختبارات الجديدة ذات معنى؟ اختبار ناجح لا يثبت شيئًا أسوأ من عدم وجود اختبار أصلاً.
إعادة العمل
لديك ثلاث خيارات، بترتيب تزايد التكلفة:
- أضف تعليقًا واطلب إعادة العمل — تُعاد المهمة إلى عامل التطوير الآلي مع ملاحظاتك، مع الحفاظ على سياقها. الأفضل للحالات التي يكون فيها العمل صحيحًا لكنه غير كامل.
- ارفض الخطة وأعد التشغيل — عندما يكون المنهج خاطئًا وليس التنفيذ.
- إلغاء المهمة — عندما لا ينبغي تنفيذ العمل أصلًا. الإلغاء يوقف العملية ويحرر مساحة العامل الآلي.
كن دقيقًا في تعليقات إعادة العمل كما لو كنت تتحدث مع زميل. "هذا لا يتعامل مع الحالة الفارغة في parseRange" ينتج عنه تصحيح؛ بينما "بحاجة إلى تحسين" يؤدي إلى تخمينات غير دقيقة.
الدمج
قم بالدمج عبر مزودك المعتاد، باستخدام قواعد الحماية والاختبارات والموافقات المعتادة. يفتح Coroid طلب السحب (pull request)؛ وعمليتك الحالية هي التي تحدد ما يحدث له.
لا يوجد شيء مميز في عملية الدمج — وهذا مقصود. pull request هو طلب سحب عادي، لذا يمر عبر نظام المراجعة واختبارات التكامل المستمر الذي تثق به.
التالي
إلى أين ننتقل الآن — يوجهك إلى باقي الوثائق حسب ما تريد القيام به.