GitHub Stacked PRs: Cứu rỗi linh hồn những gã thợ code lười chia nhỏ Task

Chào anh em đồng đạo. Bao nhiêu lần các ông ném một cái PR (Pull Request) +10,000 dòng code cho thằng lead và bị nó chửi như tát nước vào mặt rồi? Trả lời thật đi.

Hôm nay lướt Hacker News thấy quả bài về GitHub Stacked PRs (gh-stack) đang bú hẳn 681 điểm, dân tình bàn tán xôn xao. Với tư cách là một thằng từng phải review những cái PR to bằng cả cái hệ mặt trời, tôi thấy phải lôi ra đây chém gió với anh em ngay và luôn.

Nguồn cơn sự sung sướng: Rốt cuộc Stacked PRs là cái đ*o gì?

Cho những anh em tối cổ hoặc mới ra trường chưa bị đời vùi dập: Stacked PRs là một khái niệm đ*o có gì mới. Thay vì tọng một cục code chà bá lửa vào một cái PR duy nhất khiến thằng reviewer mờ mắt, cắn RAM não rồi nhắm mắt ấn Approve cho xong chuyện, thì các ông chia nhỏ nó ra.

Nhưng điểm ăn tiền là: Các PR nhỏ này được "xếp chồng" (stacked) lên nhau. PR B dựa trên code của PR A, PR C lại dựa trên PR B.

Trước đây, làm trò này trên Git thuần hoặc GitHub bình thường là một cơn ác mộng. Tưởng tượng PR A có thay đổi, ông phải quay xe rebase đống B, C, D... vỡ mồm rụng răng vì conflict. Nhận thấy sự đau khổ đó, GitHub đã chính thức tung ra gh-stack - công cụ command line (CLI) để anh em quản lý mớ bòng bong này một cách mượt mà hơn.

Dân tình Hacker News đang tế sống hay tung hô?

Chủ đề này đang kéo một lượng tương tác ảo ma trên HN. Vì không có gì hoàn hảo, nên anh em chia phe combat cực gắt:

• Phe "Đội GitHub lên đầu": Đa số anh em đều hú hét sung sướng. Cuối cùng thì cũng có hàng chính chủ. Đ*o cần phải cài cắm mấy cái tool ngoài ngoằn ngoèo nữa. Lệnh CLI nghe đồn khá mượt, giúp tự động rebase cả chuỗi stack khi có một node ở giữa thay đổi.
• Phe "Lão quái soi mói": Mấy trưởng lão từng xài Phabricator hay Graphite thì bĩu môi: "Ui dời, bọn tao xài cái workflow này từ thời cởi truồng tắm mưa rồi. Giờ GitHub mới lẹt đẹt chạy theo, đúng là chậm như rùa."
• Phe "Tâm linh bấn loạn": Có một số pháp sư ẩn danh lại lo ngại sâu sắc về vấn đề Merge Conflict. "Thế lỡ tôi đổi 1 hàm ở PR gốc, xong conflict từ trên xuống dưới cái stack thì cái tool này có tự fix không, hay lại sập server não người?"

Góc nhìn từ Coding4Food: Bài học sinh tồn chốn công sở

Tóm cái váy lại, gh-stack ra mắt là một tín hiệu đáng mừng. Mấy tính năng công nghệ đôi khi lùa gà thật, nhưng cái này thì tôi đánh giá cao.

Với vãn bối tui, bài học rút ra ở đây đ*o phải là dùng tool gì cho ngầu, mà là Sự thấu cảm với đồng nghiệp.

Các ông ạ, đừng bắt ai phải đọc một cái PR đổi 50 file với hàng ngàn dòng code. Khóa mõm, cấm chat, thậm chí unfriend nhau ngoài đời cũng từ mấy cái PR khổng lồ mà ra đấy. Hãy tập thói quen tư duy chia nhỏ vấn đề, làm từng bước, review từng phần. Code ngon, review nhanh, thì anh em mới có thời gian rảnh mà ra quán net cày game hay ngồi cà phê chém gió.

Còn chần chừ gì nữa, tải về test thử xem có ngon như lời đồn không, hay lại toang hoác.

Nguồn tham khảo:

• Hacker News: GitHub Stacked PRs