Khi codebase review có khả năng thành chiến tranh thế giới

Sao merge code không gửi anh review?

– Em thấy không có gì nghiêm trọng…

→ 1 tuần sau: production lỗi, blame nhau cả công ty

Tôi vào hỗ trợ một team nhỏ, thấy PR toàn merge trực tiếp, không ai comment gì, hoặc comment xong rồi… vẫn merge cho xong.

Không có owner rõ ràng, không CI test tự động, không rule lint – chỉ có niềm tin mù quáng vào dev trước đó.

🎯 Bài học:

  •   Code review là quy trình bảo vệ cả team – không phải để “bắt lỗi nhau”
  •   Có guideline rõ, assign reviewer, CI check trước khi review
  •   Cần dạy kỹ năng “review không toxic, không thụ động”

Post a Comment

Mới hơn Cũ hơn