Top Tweets for #reviewing_code

7 days ago

ا #reviewing_code المتعارف عليه إن الـcode reviewer هو اللي يـresolve الـcomments. هي حاجة بسيطة، بس كان عندي السؤال ده لما بدأت. لما أ-apply a code review comment هل أعمل له resolve ولا لأ، والإجابة لأ. لما بخلص خالص بعلق على الـcomment إنه Done، أو بـrequest re-review أو ببلغ الـreviewer وهو بيشوف الجديد وبيعدي عالـcomments ويـresolveـها.

Hossam El-Deen

@hossameldeenfci

7 days ago

ا #reviewing_code في نفس الموضوع محتاج تحسب "تكلفة التعليق" في مقابل قيمته. يعني لو تعليق العائد منه قليل بس تكلفة عملانه كبيرة (المبرمج هيحتاج يغير الـapproach كله) يبقى بلاش منه. لكن لو العائد منه كبير ويستحق وضروري خلاص.

Hossam El-Deen

@hossameldeenfci

7 days ago

ا #reviewing_code وانت بـreview a PR محتاج تفكر في تكلفة الـcycle وهل التعليقات اللي عايز تضيفها تستحق ولا لأ. يعني لو كل اللي عندك تعليق أو تعليقين "nitpicks" نسبياً ممكن الموضوع مايستحقش إنك تطلب أي تعديلات. لكن لو مثلاً عندك تعليق قوي، ممكن تضيف عليه التعليقين طالما كده كده cycle جديدة هتتعمل. والموضوع ده بيزيد كمان كل ما كانت مش أول cycle أو طلب تعديلات .. لإن أي cycle جديدة بتأخر طلوع الـfeature لإن صاحب الـPR ممكن يكون مسك في ticket تانية ولسه هيرجع لك تاني يوم ولا كمان كام يوم وانت كذلك.

Hossam El-Deen

@hossameldeenfci

7 days ago

ا #reviewing_code وانت بـreview a PR محتاج تفكر في تكلفة الـcycle وهل التعليقات اللي عاي�� تضيفها تستحق ولا لأ. يعني لو كل اللي عندك تعليق أو تعليقين "nitpicks" نسبياً ممكن الموضوع مايستحقش إنك تطلب أي تعديلات. لكن لو مثلاً عندك تعليق قوي، ممكن تضيف عليه التعليقين طالما كده كده cycle جديدة هتتعمل. والموضوع ده بيزيد كمان كل ما كانت مش أول cycle أو طلب تعديلات .. لإن أي cycle جديدة بتأخر طلوع الـfeature لإن صاحب الـPR ممكن يكون مسك في ticket تانية ولسه هيرجع لك تاني يوم ولا كمان كام يوم وانت كذلك.

Hossam El-Deen

@hossameldeenfci

7 days ago

ا #reviewing_code س: يعني إيه code review أصلاً؟ ج: في شركات كتير، حالياً فيه practice مشهور إن أي كود لازم يتراجع عليه من شخص واحد على الأقل (غير اللي كاتبه). المراجعة دي، في اللي أنا شفته، مش بالضرورة خالص (بل ربما في الأغلب، بس أنا خبرتي قليلة) .. مش بالضرورة خالص تبقى مراجعة مدير لمرؤسه .. دي بتبقى مراجعة زميل لزميله. (يتبع)

Hossam El-Deen

@hossameldeenfci

7 days ago

ا #reviewing_code #jetbrains وأنا بـreview a PR بحب أ-reviewـه من خلال الـIDE، زي في @jetbrains مثلاً. ده بيخليني أقدر أشوف كل الملفات اللي في الـbranch لما أدوس على الـedit icon، بدل ماشوف الملفات اللي في الـdiff بس. (بفرض إني عامل checkout للـbranch) حاجة كمان. (يتبع)

Hossam El-Deen

@hossameldeenfci

7 days ago

ا #reviewing_code - دي الخطوات اللي بمشي عليها وأنا بـreview PR (حالياً): (1) بـcheckout الـbranch locally وبفتح claude code session عشان لو عزت أسأل في حتة مش فاهمها في الكود (أو أي حاجة بفكر فيها أو عايز أتأكد منها). (2) بقرا الـticket وبشوف الدزاين وبفهم ال��requirements. (3) (اختياري) ممكن أشغل البرنامج لو عايز أتأكد إني فاهم صح أو لو فيه حتة مش متأكد متهندلة إزاي. مش هدفي إني أ-QA خالص. (4) بقرا الكود سطر سطر وبضيف تعليقاتي (طبعاً ده فيه تفاصيل كتير ممكن أبقى أقولها في بوستات بعد كده) (5) بطلب من claude code إنه يـreview (حالياً، في سيشن مختلفة عن اللي فاتحها في الأصل) وبراجع تعليقاته وبشوف لو فيه حاجة أضافها أو ماخدتش بالي منها وبضيفها في التعليقات. ساعات كتير بيلقط حاجات كويسة.

Top Tweets for #reviewing_code

Last Seen Hashtags on Sotwe

Trends for you

Most Popular Users