В мелких изменениях, и ревьюях на каждое изменение. То есть, идея частых коммитов мне тоже нравится, но мне нравится их делать на рабочую ветку (или ствол), который до полного релиза не виден пользователям. А ревьюи по уму надо делать на большие законченные фичи. Если их делать на каждое мелкое изменение, то волей-неволей ревьюер следует за ходом мысли изначального автора и не видит большую картину. То есть, если ставить целью ревьюев поиск неправильно поставленных пробелов - то да, так проще. Но это плохие, негодные ревьюи. А если ставить целью оценку правильности всей фичи и мыслепоиск глюков в ней, то надо рвеьюить большие куски кода сразу, разглядывая всю суету с дистанции.
![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png)
timeasmymeasure
no subject
Date: 2012-03-18 05:13 pm (UTC)no subject
Date: 2012-03-19 04:03 pm (UTC)no subject
Date: 2012-03-20 07:28 am (UTC)Не зря же изменения делаются маленькими.
А как следует проводить code review: в одиночистве или в диалоге с писателем?
no subject
Date: 2012-03-20 03:33 pm (UTC)СобÑÑвенно, пÑоблема в маленÑÐºÐ¸Ñ Ð¸Ð·Ð¼ÐµÐ½ÐµÐ½Ð¸ÑÑ ÐºÐ°Ðº Ñаз в Ñом, ÑÑо за деÑалÑми ÑеÑÑеÑÑÑ Ð±Ð¾Ð»ÑÑÐ°Ñ ÐºÐ°ÑÑина, и запÑоÑÑо можно поÑиниÑÑ Ð¾Ð´Ð½Ð¾ меÑÑо и пÑи ÑÑом иÑпоÑÑиÑÑ Ð´ÐµÑÑÑÑ Ð´ÑÑÐ³Ð¸Ñ . ÐÐ¾Ñ Ð´Ð»Ñ ÑÑого по ÑÐ¼Ñ Ð¸ нÑÐ¶Ð½Ñ ÑевÑÑи: ÑÑоб оÑойÑи на некое ÑаÑÑÑоÑние и поÑмоÑÑеÑÑ Ñо ÑÑоÑонÑ, впиÑÑваÑÑÑÑ Ð»Ð¸ Ð¸Ð·Ð¼ÐµÐ½ÐµÐ½Ð¸Ñ Ð² болÑÑÑÑ ÐºÐ°ÑÑинÑ. Рне пÑоÑивоÑеÑÐ°Ñ Ð»Ð¸ дÑÑг дÑÑÐ³Ñ ÑаÑÑи ÑезÑлÑÑаÑа.
ÐÑ Ð´Ð°, ÑеÑÑÑ ÑÑÑ ÐºÐ¾Ð½ÐµÑно Ñоже в помоÑÑ, но Ñ ÑеÑÑов дÑÑÐ³Ð°Ñ ÑÑнкÑиÑ: они ловÑÑ ÑовÑем гÑÑбÑе поломки, и более Ñого, заÑанее пÑедÑказаннÑе гÑÑбÑе поломки. ФÑнкÑÐ¸Ñ ÑевÑÑÑ - в Ñом, ÑÑÐ¾Ð±Ñ Ð½Ðµ дÑблиÑоваÑÑ Ñобой ÑеÑÑÑ, а дополнÑÑÑ Ð¸Ñ . ÐоддеÑживаÑÑ ÐºÐ¾Ð½ÑиÑÑенÑноÑÑÑ Ð±Ð¾Ð»ÑÑой каÑÑинÑ. ÐÑкаÑÑ Ð²Ð¾Ð·Ð¼Ð¾Ð¶Ð½Ð¾ÑÑи новÑÑ Ð¿Ð¾Ð»Ð¾Ð¼Ð¾Ðº, коÑоÑÑе не пÑедÑÐºÐ°Ð·Ð°Ð½Ñ Ð² ÑÑаÑÑÑ ÑеÑÑÐ°Ñ , и ÑооÑвеÑÑÑвенно добавлÑÑÑ Ð´Ð»Ñ Ð½Ð¸Ñ Ð½Ð¾Ð²Ñе ÑеÑÑÑ.
ÐÑоводиÑÑ ÑевÑÑй ÑледÑÐµÑ ÑнаÑала ÑÑением в одиноÑеÑÑве, а поÑом в диалоге. ÐÑли неÑÑо непонÑÑно пÑи ÑÑении в одиноÑеÑÑве, Ñо знаÑÐ¸Ñ Ð¾Ð½Ð¾ или Ð¿Ð»Ð¾Ñ Ð¾ докÑменÑиÑовано и ÑÑда надо добавиÑÑ ÐºÐ¾Ð¼Ð¼ÐµÐ½ÑаÑий, или пÑоÑÑо кÑиво и должно бÑÑÑ Ð¸ÑпÑавлено.
no subject
Date: 2012-03-20 08:13 pm (UTC)Ðа мой взглÑд диалог пÑи code review полезен Ñем, ÑÑо позволÑÐµÑ Ð¿ÑовеÑÑи review в ÑÐ°Ð·Ñ Ð±ÑÑÑÑее.
ÐÑÑÑÑее обÑÑдиÑÑ Ð°Ð»ÑÑеÑнаÑивнÑе Ð¿Ð¾Ð´Ñ Ð¾Ð´Ñ Ðº напиÑÐ°Ð½Ð¸Ñ ÐºÐ¾Ð´Ð°.
ÐÑÑÑÑее обÑÑдиÑÑ Ð²Ð¾Ð·Ð¼Ð¾Ð¶Ð½Ñе подводнÑе камни и где именно код Ð¸Ñ Ð¾Ð±Ñ Ð¾Ð´Ð¸Ñ.
ÐÑÑÑÑее воÑÑоздаÑÑ ÐºÐ¾Ð½ÑекÑÑ Ð² голове пÑовеÑÑÑÑего.
2) ÐейÑÑвиÑелÑно: еÑли непонÑÑно из кода, ÑÑо пÑоиÑÑ Ð¾Ð´Ð¸Ñ, Ñо код, веÑоÑÑно, нÑжно пеÑепиÑÑваÑÑ.