AE
Обидно, когда написал идеальный код, а тебе замечания какие-то пишут в комменты к пул реквесту
Систему оценки мр используете?
Size: a a a
2020 July 13
SL
а должны были восхищение высказывать
Пользовалась когда-нибудь лайками в пул реквестах?
SL
Было два случая, или даже три, когда чел на испытательном увольнялся после адекватных комментов к мр, называя их хернёй
У меня коллега обижался на комменты)
Потом на корпоративе выпили и обсудили)
Потом на корпоративе выпили и обсудили)
SL
Систему оценки мр используете?
Не-а. А что за оценка?
LV
Пользовалась когда-нибудь лайками в пул реквестах?
В нашей системе поставить лайк — это одобрить
SL
В нашей системе поставить лайк — это одобрить
У нас можно комменты лайкать к пул реквестам
SL
В нашей системе поставить лайк — это одобрить
А что за система?
AE
Не-а. А что за оценка?
Есть два ревьювера и сонар, каждый из них может оценить на -2, -1, 0, +1. Мр сливается автоматом, когда у него оценка +2, с оценкой +1 может слить тимлид. Баллы суммируются
LV
У нас можно комменты лайкать к пул реквестам
прикол. Не, у нас нельзя
AE
Есть два ревьювера и сонар, каждый из них может оценить на -2, -1, 0, +1. Мр сливается автоматом, когда у него оценка +2, с оценкой +1 может слить тимлид. Баллы суммируются
Оценку должны дать все трое
AE
У нас можно комменты лайкать к пул реквестам
В гитлабе, ага
AE
Есть два ревьювера и сонар, каждый из них может оценить на -2, -1, 0, +1. Мр сливается автоматом, когда у него оценка +2, с оценкой +1 может слить тимлид. Баллы суммируются
Переоценка сонаром идёт только после коммита в бранч, оценивается результат мерджа
SL
Оценку должны дать все трое
Это как-то интегрировано в систему мерж реквестов или руками в комменте пишете?
AE
После коммита оценка людей не сбрасывается
AE
Это как-то интегрировано в систему мерж реквестов или руками в комменте пишете?
Да, интегрировано, было такое на одном проекте, название инструмента не вспомню, года три прошло
AE
@dmsol не помнишь такой?
SL
В гитлабе, ага
У нас bitbucket