الأهداف
ولعملية المراجعة عدة أهداف:
- ضمان جودة عالية للرموز البرمجية، من حيث الوظائف وسهولة القراءة
- احصَل على الأخطاء بسبب حدوث الأخطاء.
- الحفاظ على أسلوب متّسق لتسهيل بدء العمل في أي جزء من قاعدة الترميز
تخضع جميع الرموز البرمجية التي يتم إدخالها في blockly- sample و core Blockly للمراجعة، سواء كتبها مساهمون في المنتدى أو أعضاء فريق Blockly.
وبصفتنا مراجعين، نهدف إلى العمل معك لتحقيق أفضل نتيجة ممكنة. بصفتك من المساهمين، نطلب منك التواصل معنا لإجراء محادثة حول طلبات السحب من أجل مراجعة طلباتك ودمجها.
العملية
تمر عملية مراجعة العلاقات العامة بعدة مراحل:
Assignment
عند تلقّي طلب السحب، يعيّن العضو قيد الطلب في فريق Blockly مراجعًا.
يتم اختيار المراجعين بناءً على الخبرة ولتوزيع أعباء العمل بالتساوي.
قد يستغرق تعيين مُراجع بضعة أيام، كما قد يستغرق بضعة أيام أخرى للحصول على مراجعة. لا داعي للقلق، فهذا أمر طبيعي.
إضافة ملاحظات
خلال مرحلة الملاحظات، يقدم المراجع اقتراحات بشأن التغييرات في العلاقات العامة. قد تكون هذه إجراءات بسيطة لجعل الرمز الخاص بك متوافقًا مع دليل أسلوب JavaScript من Google. أو يمكن أن تكون أشياء أكبر مثل مطالبتك بإعادة تنظيم تعريفات الدوال.
نشجّع المراجعين على استخدام مراجعات الرموز البرمجية من GitHub (بدلاً من إضافة تعليقات فردية) لكي تتلقّى إشعارًا واحدًا بدلاً من عدة إشعارات.
مناقشة
مرحلة المناقشة هي فرصتك للرد على الملاحظات. ربما لم يكن أحد تعليقات المراجعة واضحًا: الآن فرصتك لطلب التوضيح. أو ربما طلب المراجع تغييرًا، ولكنك تعتقد أنه سيكون له تداعيات: الآن فرصتك للتوصل إلى حل وسط.
التغيير
مرحلة المراجعة هي المكان الذي يمكنك فيه إجراء تغييرات على العلاقات العامة. عادةً ما تكون هذه التغييرات نتيجة لما قاله المراجع في مرحلة الملاحظات.
بعد الانتهاء من المراجعات، قد يكون من المفيد الإشارة إلى المراجع لتطلب منه إلقاء نظرة أخرى.
التكرار
بعد مرحلة المراجعة، تتاح للمراجع فرصة أخرى لتقديم ملاحظات، وتبدأ العملية من البداية.
غالبًا ما تكون المراجعة الثانية بسيطة وتركز على وحدات القياس مثل علامات الترقيم وأسلوب التعليمات البرمجية. لكن في بعض الأحيان يمكن أن تكون المراجعة الثانية كبيرة جدًا. قد يطلب المراجع الأول من شخص آخر إلقاء نظرة، للحصول على منظور جديد.
الدمج!
مرحلة الدمج هي فرصتك للاحتفال. لقد قمت بإنشاء تغيير، وناقشته وراجعته، وأخيرًا دمجته! هذا إنجاز عظيم لا يبدأه الكثير من الأشخاص، ناهيك عن إكماله!
نشكرك على كل ما بذلته من جهد لتحسين تطبيق Bluely. وتهانينا!