مراجعة الكود بالإنجليزية
طرح الملاحظات بلطف، التمييز بين الاقتراح والاعتراض، وقبول النقد.
ثقافة المراجعة الأمريكية تفصل الشخص عن الكود: الملاحظة توجّه إلى السطر لا إلى الكاتب. والاصطلاحات شائعة: nit ملاحظة تجميلية صغيرة، وblocking اعتراض يمنع الدمج، وLGTM موافقة.
- 1
Small nit, maybe rename this variable.
ملاحظة صغيرة جداً، غيّر اسم المتغير.
- 2
Just a suggestion, not blocking.
هذا اقتراح فقط، ليس اعتراضاً.
- 3
I think this could be simplified a bit.
أظن أن هذا يمكن تبسيطه قليلاً.
- 4
What do you think about using a map here instead?
ما رأيك لو استخدمنا خريطة بدلاً منها؟
- 5
Is there a reason for doing it this way?
هل هناك سبب معين لهذا الاختيار؟
- 6
I might be wrong, but this looks duplicated.
قد أكون مخطئاً، لكن هذا يبدو مكرراً.
- 7
This part is a little hard to follow for me.
هذا الجزء صعب القراءة في نظري.
- 8
I'd like to see a test for this case.
أرى أن هذا يحتاج اختباراً إضافياً.
- 9
This will break if the list comes back empty.
هذا سيفشل إذا كانت القائمة فارغة.
- 10
This one's blocking, I'm afraid.
هذا اعتراض حقيقي عليّ توضيحه.
- 11
Let's hash this out on a call.
لنتحدث عنه في مكالمة أوضح.
- 12
Good catch, I didn't think of that.
نقطة جيدة، لم أفكر فيها.
- 13
You're right, I'll fix it.
أنت محق تماماً، سأعدّلها.
- 14
I'd rather keep it as is, and here's why.
أفضّل إبقاءه كما هو لهذا السبب.
- 15
Let's open a ticket for that later.
لنسجّل هذا كمهمة لاحقة.
- 16
That's out of scope for this PR.
هذا خارج نطاق هذه المراجعة.
- 17
I'll handle it in a follow-up PR.
سأتابعه في طلب دمج منفصل.
- 18
This looks good to me overall, nice work.
كل شيء يبدو جيداً بالنسبة لي.
- 19
Approved on my side, feel free to merge it.
وافقت عليه، ادمجه متى شئت.
- 20
Thanks a lot for the thorough review on this.
شكراً على المراجعة الدقيقة والشاملة.