متى تتوقف وتتحقق — self-review قبل push
حتى الآن راجعنا PR مفتوحًا بالفعل، بعد أن يكون CI قد جرى. في هذا الدرس نعود خطوة إلى الوراء، إلى اللحظة التي تسبق حتى الـ push: اللحظة التي ينتهي فيها الوكيل من تحرير الشيفرة، وتكون أنت ما زلت وحدك أمام الـ diff، قبل أن يصبح مشكلة شخص آخر. الـ self-review هو عادة قراءة الـ diff الكامل، وتشغيل الاختب
قبل مغادرة المنزل بحقيبة مُعبأة، تفتحها مجددًا وتتحقق بنفسك مما بداخلها فعلًا. لا يكفي أن يكون السحّاب مغلقًا وأن تبدو مرتبة من الخارج — عليك التأكد من أن شيئًا لم يدخل إليها دون أن تضعه أنت عن قصد.
- المراجعة الذاتية (self-review) قبل push
- عادة قراءة الـ diff الكامل وتشغيل الفحوصات المحلية قبل دفع الشيفرة، لالتقاط مشكلة قبل أن تصبح PR عامًا يضطر شخص آخر لكشفه.
- git diff --stat
- نظرة سريعة على كل ملف تغيّر وعدد الأسطر التي تغيّرت في كل منها — الخطوة الأولى في الـ self-review، قبل الغوص في تفاصيل أي ملف بمفرده.
- أثر جانبي صامت
- تعديل يظهر في نفس الـ diff لكنه غير متعلق بالمهمة المطلوبة؛ يمرّ عبر CI بصمت لأن لا اختبار قائم يفحصه، وقراءة الـ diff نفسه وحدها تكشفه.
- فحص محلي قبل push
- تشغيل الـ lint والاختبارات على جهازك الخاص قبل الدفع، لالتقاط المشكلات في أرخص لحظة ممكنة لإصلاحها — قبل أن تصبح ملاحظات عامة على PR مفتوح.
