Kỹ sư DevOps · Bước 3/8 · 8 phút
Git và review code
Dùng branch, commit và pull request như các đội thật sự làm, và review thay đổi một cách hữu ích.
Git là sổ ghi mọi thay đổi
Git theo dõi mọi thay đổi của code và cấu hình: ai đổi gì, lúc nào và vì sao. Trong DevOps, mọi thứ định nghĩa nên một hệ thống đều nên nằm trong Git, kể cả file hạ tầng và pipeline. Thứ gì không có trong Git thì khó review, khó làm lại và khó quay lui.
Vòng lặp hằng ngày
Clone repository, tạo branch, sửa một chút, commit kèm lời mô tả rõ ràng, push lên và mở pull request (yêu cầu gộp code). Người khác review, các kiểm tra tự động chạy, rồi thay đổi được merge vào nhánh chính.
Giữ commit nhỏ và lời mô tả có ích: đổi gì và vì sao, đừng chỉ ghi fix. Học git status, git diff, git log, và cách xử lý merge conflict một cách bình tĩnh.
Review cho tốt
Review code bắt lỗi, lan toả kiến thức và giữ tiêu chuẩn nhất quán. Một lần review tốt sẽ hỏi: thay đổi có làm đúng điều nó nói không, cái gì có thể hỏng, đã có test chưa, và một năm sau người khác có chạy được không.
Hãy tử tế và cụ thể. Nhận xét về code, không nhận xét về con người. Pull request nhỏ được review kỹ hơn; một thay đổi 2.000 dòng thường chỉ nhận cái gật đầu nhanh chứ không ai review thật.
Bảo vệ nhánh chính
Phần lớn các đội yêu cầu có review và các kiểm tra đều qua trước khi merge vào main. Hãy bật chế độ này cho cả dự án của riêng bạn. Không bao giờ commit mật khẩu hay key; một khi đã push, hãy coi như chúng đã công khai và thu hồi ngay.
Thử ngay
Tạo và review pull request đầu tiên
- 1.Tạo tài khoản GitHub hoặc GitLab miễn phí và một repository công khai mới có README.
- 2.Cài Git trên máy, clone repository và tạo một branch.
- 3.Sửa README, commit với lời mô tả rõ ràng, push và mở pull request.
- 4.Đọc diff của chính mình như một người review và để lại một nhận xét trước khi merge.
Đây là diff trong pull request của tôi: [dán diff, không kèm bí mật]. Hãy review như một kỹ sư senior cẩn thận. Liệt kê rủi ro, test còn thiếu và tên đặt chưa rõ, theo thứ tự quan trọng.