الإرشاد و الـ Code Review
الجزء اللي مش هتلاقيه في أي حتة تانية: مهندس خبير بيقرا شغلك إنت ويقولك رأيه فيه.
- مانع
إيه اللي بيحصل لو المزود عمل timeout هنا؟ دلوقتي المستدعي بياخد promise ناجح من غير بيانات، وده مش مختلف عن نتيجة فاضية.
- اقتراح
منطق الـ retry والتحليل بيعملوا وظيفتين في function واحدة. لو فصلتهم هتقدر تختبر الـ retry من غير fixture.
- ملاحظة صغيرة
التسمية — `data2` مش هتفكرك بحاجة بعد ست شهور.
أول رد خلال يوم عمل واحد. وكل تعليق بيوضح لو كان مانع للدمج.
الإرشاد و الـ Code Review
اهتمام مهندس كبير بشغلك إنت
ده الجزء اللي مش هتاخده من درس ولا كورس ولا مساعد ذكاء اصطناعي بيوافقك على كل حاجة. حد خبرته أكبر بيقرا اللي كتبته ويقولك رأيه فيه.
- Code review
- سطر بسطر، على كل pull request، ومعاها السبب.
- ملاحظات معمارية
- هل تصميمك متوافق مع النظام ولا بيصارعه.
- ملاحظات هندسية
- إنت بتشتغل إزاي، مش بس إنتجت إيه.
- توجيه في القرارات
- المهندس الخبير بيختار إزاي بين حلين الاتنين منطقيين.
إيه اللي هنقوله وإيه اللي مش هنقوله
لسه مفيش آراء أعضاء، لأن لسه مفيش أعضاء
برمج جديد. بدل ما نستعير مصداقية مكسبناهاش، ده بالظبط اللي بنلتزم بيه — وتقدر تحاسبنا عليه.
رد من إنسان خلال يوم عمل واحد
محسوب من لحظة ما الـ pull request يبقى جاهز لمراجع، بساعات العمل. ولما طاقة المراجعين تخلص بنوقف قبول أعضاء جداد بدل ما نسيب سرعة الرد تقل.
كل كلمة بنقولها عنك ينفع تتشرح
لو الـ mentor مش قادر يشرحلك دليل معين في جملة واحدة، الدليل ده مابيتكتبش.
مفيش ترتيب بين الأعضاء
مفيش لوحات صدارة، ولا عدّ commits، ولا سطور كود. الأرقام دي بتكافئ سلوك غلط، وإحنا مابنجمعهاش كدرجات أصلًا.
شغلك يفضل شغلك، على حساب GitHub بتاعك
المساهمات بتتم على حسابك إنت، يعني اللي بتبنيه ظاهر ويتأكد منه من غير ما حد ياخد كلامنا كده.
لما الأعضاء يخلّصوا أول مشاريعهم هننشر اللي حصل فعلًا، بما فيه اللي ما نجحش — بإذنهم، ومن غير أسماء لو طلبوا كده.
إنت مش محتاج درس تاني.
إنت محتاج خبرة.
الطلبات بيقراها إنسان، والأماكن محدودة بعدد الـ pull requests اللي نقدر نراجعها صح كل أسبوع.
قدّم على برمج