Yêu cầu đối với reviewer

  • Có trách nhiệm kiểm soát coding convention, issue redmine đối với merge request, trường hợp vi phạm cần thông báo các vấn đề cho developer.
  • Có trách nhiệm đọc hiểu code ở mức ý tưởng và giải pháp, trường hợp không hiểu cần liên hệ với developer để tìm hiểu. Tuyệt đối không merge các request không rõ ý tưởng.
  • Có trách nhiệm hướng dẫn developer trong các trường hợp sau:
    • Có cách viết code tốt, rõ ràng hơn.
    • Đã có các đoạn code dùng chung, tiến hành dùng lại.
    • Những đoạn code có khả năng dùng chung, tiến hành tách code.