Khi review code, bạn có mẹo riêng nào để nhận ra tốt hơn những thứ “đáng lẽ phải thay đổi nhưng lại chưa được thay đổi” không?

Với những "thay đổi" xảy ra trong commit, PR hoặc MR lần này, thì việc để lại bình luận xem thay đổi đó ổn hay không là khá dễ. Nhưng việc nhận ra những phần đáng lẽ phải được thay đổi mà lại chưa được thay đổi rồi để lại bình luận về chúng, thì dù đã review code trong một thời gian khá dài, tôi vẫn thấy điều đó khó. Nếu đó không phải là codebase mà tôi cực kỳ quen thuộc thì lại càng như vậy.

Tôi cũng nghĩ ra được vài cách, nhưng sẽ tốt hơn nếu có thêm nhiều cơ chế hỗ trợ. Tôi muốn biết mọi người đã thử làm như thế nào.

• Khi triển khai: thiết kế sao cho những phần cần được thay đổi cùng nhau về sau có thể được mã kiểm thử bao phủ cùng nhau.
• Khi viết PR: thêm vào mẫu PR những câu hỏi phù hợp để reviewee cân nhắc. (ví dụ: Có PR nào trước đây có mục đích tương tự PR lần này không? Khi so sánh với PR đó, liệu trong PR lần này có phần nào bị bỏ sót không?)
  • Mỗi khi trong codebase đó xảy ra trường hợp "đáng lẽ phải làm nhưng lại bỏ sót", hãy ghi lại, rồi mỗi lần như vậy lại cập nhật mẫu PR bằng ví dụ tương ứng.
• Khi review PR: thêm vào hướng dẫn review để reviewer cũng tìm những phần bị thiếu theo cách tương tự các câu hỏi ở trên.