Tại sao code review quan trọng?
1. Code Review là gì?
Code review là một thuật ngữ chung chỉ quy trình trong đó các thay đổi về code sẽ được người khác kiểm tra trước khi được tích hợp vào codebase của dự án.
Các thuật ngữ pull request và merge request có liên quan mật thiết đến code review, nhưng thường bao hàm cả hành động merging.
2. Tại sao cần sử dụng Code Review?
Như chúng ta đã học trong bài trước về Feature Branching Workflow, các Git repository thường là bước đầu tiên trong một pipeline (quy trình) mà đích đến cuối cùng là code được triển khai lên máy chủ production.
Phần mềm hiện đại thường phức tạp và có tính liên kết chặt chẽ. Code mới viết ra có thể chứa lỗi sai, hoặc vô tình gây ảnh hưởng xấu đến các khu vực khác của codebase. Để giảm thiểu rủi ro xảy ra sự cố và cải thiện chất lượng code, một thói quen tốt là nhờ đồng nghiệp review các thay đổi code mới trước khi chúng được merge.
3. Quy trình Code Review
Một quy trình code review thông thường trong các nhóm phát triển phần mềm thường diễn ra như sau:
-
Các thành viên trong nhóm làm việc trên những branch tách biệt để phát triển tính năng hoặc sửa lỗi. Khi hoàn thành một thay đổi, developer sẽ tạo một pull request hoặc merge request thay vì trực tiếp push vào
main. -
Pull request này sẽ tóm tắt các thay đổi, giải thích mục đích hoặc ảnh hưởng của chúng, và gửi tới các reviewer để đánh giá.
-
Quá trình review có thể bao gồm nhận xét, góp ý, hoặc yêu cầu sửa đổi từ các reviewer. Người tạo yêu cầu sẽ chỉnh sửa lại code dựa trên phản hồi nhận được. Những vòng sửa chữa và phản hồi này có thể lặp lại cho tới khi mọi comment được giải quyết hoặc tất cả reviewer đồng ý rằng code đã đạt chất lượng.
-
Chỉ khi nhận được sự chấp thuận từ các reviewer, thay đổi mới sẽ được merge vào branch chính của dự án để chuẩn bị triển khai hoặc tiếp tục phát triển.
Tổng kết
Code review là một quy trình quan trọng để đảm bảo chất lượng code và giảm thiểu rủi ro xảy ra sự cố. Nó giúp các developer tìm ra lỗi và vấn đề trong code một cách sớm và hiệu quả.
Tiếp theo: Code review vs Pull requests vs Merge requests