----- Original Message ----- > From: "Duong \"Yang\" Ha Nguyen" <[email protected]> > To: "Hanoi Linux Users Group" <[email protected]> > Sent: Saturday, February 26, 2011 9:23:12 AM > Subject: Re: [HanoiLUG] Ubuntu "ra riêng" > On Sat, 26 Feb 2011, Nguyen Vu Hung (VNC) wrote: > > > (2011/02/25 21:02), Phan Anh Dung wrote: > > ... > > việc coding standard chỉ là chuyện nhỏ, cái ý tưởng có được > > chấp nhận không mới là > > chuyện lớn." > > ... > > > > +1 > > > > 0 > > > > Người phải review các commit sẽ nghĩ: > > > > Ngay cả những điều nhỏ như coding standard của một dự án lớn 10M LOC > > không giữ được thì đừng nói gì đến code quality. > > > > Nếu là em, em sẽ bắt sửa cho đúng chuẩn (nghĩa là reject tạm thời) > > +1 > > Rất ủng hộ ý kiến của anh Hưng. Nếu làm một developer mà không muốn > bới lông tìm vết thì không thể nào cho ra sản phẩm tốt được. > Developer giống như một nhà quản lý sử dụng kỹ năng tư duy (business > logic) và ngôn ngữ (ngôn ngữ lập trình) của mình để chỉ đạo nhân viên > (tức computer) thi hành các công việc nào đó. Nếu ngôn ngữ diễn đạt > không hoàn thiện, không chuẩn tắc, không nhất quán, ẩn chứa sai sót > thì sao nhân viên có thể thi hành công việc tốt.
+1. Chỉ ý tưởng tốt thì không ra sản phẩm tốt được, phải bắt tay vào làm thực tế mới được. Làm FOSS là làm cùng nhiều người, mà toàn những người lạ, trình độ kỹ thuật, văn hóa khác biệt. Nếu không cùng tuân theo chuẩn thì khó lắm. Nếu ai đó chỉ có 1 ý tưởng, hãy đi nói chuyện với 1 coder/contributor bình thường của dự án FOSS đó, cùng nhau implement, cùng nhau commit lên upstream khi code đã tốt, đúng chuẩn rồi. Đội review không thể làm thay được. Bản thân tôi cũng là 1 reviewer trong Fedora. Chúng tôi được quán triệt rất rõ là nếu chưa hợp chuẩn, mark vào là chưa, chỉ rõ hơn cho người contributor được ngay thì tốt, không thì qua IRC cho nhanh; nhất quyết không sửa hộ. Một lưu ý nhỏ cho mọi người rõ thêm: Chuẩn ở đây không phải chỉ là coding standards, mà còn nhiều qui tắc khác bản thân upstream thống nhất tự đặt ra (thường ban đầu là do core team, sau đó các thành viên mới có thể đóng góp thêm). VD: chỉ riêng việc packaging trong Fedora, packager phải tự tuân theo guidelines [1]. Khi nào còn tự thấy mình chưa ok thì đừng submit vội. Còn reviewer có guidelines riêng và review theo checklist [2]. [1] http://fedoraproject.org/wiki/Packaging/Guidelines [2] http://fedoraproject.org/wiki/Packaging/ReviewGuidelines Kind regards, Tuan _______________________________________________ POST RULES : http://wiki.hanoilug.org/hanoilug:mailing_list_guidelines _______________________________________________ HanoiLUG mailing lists: http://lists.hanoilug.org/ HanoiLUG wiki: http://wiki.hanoilug.org/ HanoiLUG blog: http://blog.hanoilug.org/
