Merge mà không review – hoặc review cho có

Em bấm merge thôi chứ có ai comment gì đâu ạ… chỉ thấy nút Approve xanh lè!

PR được tạo, nhưng hoặc là không ai review, hoặc review kiểu “Ok, merge đi!” mà không đọc code. Kết quả? Bug lọt vào production, logic chồng chéo, và cả team đổ lỗi cho nhau như trong phim drama.

Mô tả: Review code chỉ để… cho có

Thực trạng:

• PR merge ngay sau khi tạo, không ai kiểm tra.
• Review hình thức: Comment “LGTM” (Looks Good To Me) dù chưa đọc hết code.
• Không có checklist, không kiểm tra logic, bảo mật, hay performance.

Hệ quả:

• Bug lọt production: Lỗi ngớ ngẩn như null pointer lọt qua.
• Code rối: Logic trùng lặp, không tối ưu, tích lũy dần thành nợ kỹ thuật.
• Không ai chịu trách nhiệm: Khi lỗi xảy ra, không rõ ai duyệt code.

Giải pháp: Enforce review nghiêm túc

Để review có ý nghĩa:

• Bắt buộc reviewer: Yêu cầu 1–2 người duyệt PR, dùng code owner để assign theo module.
• Checklist review: Kiểm tra logic, bảo mật, test coverage, naming convention.
• Tích hợp CI: Chỉ cho merge khi test pass, coverage đạt ngưỡng (ví dụ: 70%).

Ví dụ: Trong .github/CODEOWNERS, assign team backend duyệt code /api/*.

🎯 Tóm lại: Review code nghiêm túc là “lá chắn” chống bug. Bắt buộc reviewer, dùng checklist, và tích hợp CI để PR chất lượng hơn.