Mất 6 ngày để thay đổi một dòng mã (2015)

• Nhà máy đang được sử dụng dưới 10% công suất.
• Chính sách công ty hạn chế việc tạo ra backlog quá 3 tháng.
• Thay đổi chính sách thành 4 tháng sẽ giải quyết được vấn đề.
• Việc thay đổi cấu hình của phần mềm legacy đòi hỏi phải sửa một dòng mã.
• Quy trình triển khai thay đổi bao gồm gửi ticket, hoàn thành các mục bắt buộc và nhận phê duyệt của giám đốc.
• Thay đổi này là khẩn cấp để tránh bị sa thải.
• Lập trình viên đã sửa mã thành công, nhưng các biến hardcode và những lỗi khác lại gây ra vấn đề.
• Mã cần được rà soát đồng cấp và kiểm thử, sau đó mới có thể đưa lên production.
• Việc truy cập môi trường kiểm thử cần thiết bị trì hoãn do vấn đề về quyền hạn và tính sẵn có.
• Bản ghi tham số cần được đổi tên và có audit trail.
• Lập trình viên thực hiện các thay đổi cần thiết và gửi lại mã để được rà soát.
• Việc kiểm thử yêu cầu một kế hoạch kiểm thử phù hợp, bao gồm các test case do người dùng lựa chọn và kết quả kỳ vọng.
• Sau 6 ngày, chương trình được phê duyệt để đưa lên production.

[GN⁺]

Ý kiến Hacker News

• Vấn đề cốt lõi là từ chối khi người review yêu cầu các thay đổi ảnh hưởng đến những phần khác của codebase.
• Chống lại pull request được gom gọn và sự phình to phạm vi là một bài học quan trọng.
• Quy trình review code có thể đầy những lời bắt bẻ khó chịu và các bình luận vụn vặt.
• Đội bảo mật có thể không phản hồi nhanh các yêu cầu cấp quyền.
• Tiêu đề bài viết có thể gây hiểu lầm, và trong 6 ngày đó đã có thêm các cải tiến bổ sung.
• Một thay đổi chỉ một dòng có thể dẫn đến những hệ quả không ngờ.
• Quy trình review code có thể trở thành cổng kiểm soát và làm chậm tiến độ.
• Cho phép bình luận mà không chặn commit có thể dẫn đến việc phát triển hiệu quả hơn.
• Chuyển từ một đội có review code chính thức sang một đội không làm vậy có thể mang lại cảm giác mới mẻ và trao quyền.
• Có sự khác biệt trong cách quản lý công nhân nhà máy và lập trình viên phần mềm.
• Giữ lại các thay đổi dựa trên lý tưởng luôn thay đổi của cả nhóm là điều thiếu hiệu quả.
• Vấn đề nằm ở quy trình của công ty, không phải ở bản thân việc review code.