ND
что значит вердикт ревью? вы видели процесс ревью?
Size: a a a
2021 May 07
LT
критикует, потому что вы попросили глянуть, или потому что все задачи проходят ревью и опытный коллега должен их все покритиковать?
LT
что на выходе?
переделывай
норм
офигенчик
??
переделывай
норм
офигенчик
??
SM
У мидла не может ревьюит код более опытный коллега?
LT
этот вопрос куда-то в сторону
SM
Про ревью имел ввиду)
LT
мочь - может. а надо?
ND
)))
я могу оставить комменты, где подправить или где можно получше сделать. А вот за "норм" и "офигенчик" ревьюер может и огрести.
я могу оставить комменты, где подправить или где можно получше сделать. А вот за "норм" и "офигенчик" ревьюер может и огрести.
LT
эти комментарии потом исполняются? что происходит с задачей - в нее обязательно нужно сначала внести эти поправки или она проходит дальше без них?
VF
Сначала сделать, а потом подумать #всекакмылюбим
LT
ответ-то будет?))
SM
Т.е. более опытный коллега не может предложит решение лучше? Или имеется ввиду что мидл пишет более-менее приемлемое решение и этого достаточно?
ND
нужно, и обычно они занимают меньше времени, чем работа над задачей в целом
SM
Не совсем понял, какой вопрос?
VF
Почему опытный коллега не может предложить решение лучше до того, как код написан?
c
Шо ни день, то день сурка
SM
Может, ну тогда надо вводить этап обсуждения задачи
LT
насколько критичные эти поправки? без них все сломается? будет работать с натяжкой? будет работать не так, как надо? будет работать в целом норм и так, как надо, но можно было бы и лучше?
LT
называется дизайн ревью