Size: a a a

2020 May 24

AM

Artem Malyshev in rannts
Kirill (Cykooz) Kuzminykh
А раньше как их можно было искать?
Я обычно один раз смотрю пул-реквест. Ставлю все коменты. После обновления реквеста, первым делом иду по файлам где были коменты (это как раз удобнее стало, список файлов теперь видно всегда). Смотрю изменения и удаляю старые коменты, если их "исправили", делаю новые. И всё заново. Когда в пул-реквесте не останется коментов, то значит всё ОК.
Вот мне напихали комментов в PR. Я пошел их править. Запушил. Они пропали из diff. Теперь надо через activity выключить фильтры и искать их, чтобы отписать что пофиксил.
источник

KK

Kirill (Cykooz) Kuzm... in rannts
Там у каждого файла в шапке есть циферка со всеми коментами - её тыкаешь и он показывает все "скрытые" коменты. Я там и проверяю, что написал я и как исправили. И потом удаляю их от туда.
источник

KK

Kirill (Cykooz) Kuzm... in rannts
Активити я вообще не пользуюсь
источник

AM

Artem Malyshev in rannts
Kirill (Cykooz) Kuzminykh
Там у каждого файла в шапке есть циферка со всеми коментами - её тыкаешь и он показывает все "скрытые" коменты. Я там и проверяю, что написал я и как исправили. И потом удаляю их от туда.
С новым интерфейсом шапки вроде нет. Там коммиты теперь.
источник

KK

Kirill (Cykooz) Kuzm... in rannts
Хм, это я ещё не проверял
источник

VR

Vasily Ryabov in rannts
В гитлабе таких проблем нет. Просто делаешь Resolve comment и коллапсишь с глаз долой. Можно и unresolve, если передумал. К продуктам от Atlassian вообще настороженно отношусь, дизайн интерфейса мне не очень нравится. Хотя порой приходится юзать. Jira вот ещё более-менее, но тоже есть вопросы.
источник

SZ

Sergey Z in rannts
В гитлабе комент может резолвить не только ревьювер, и это ужасно.
источник

VR

Vasily Ryabov in rannts
Да, но это решается парой воспитательных бесед.
источник

in

ildar nizamov in rannts
Vasily Ryabov
В гитлабе таких проблем нет. Просто делаешь Resolve comment и коллапсишь с глаз долой. Можно и unresolve, если передумал. К продуктам от Atlassian вообще настороженно отношусь, дизайн интерфейса мне не очень нравится. Хотя порой приходится юзать. Jira вот ещё более-менее, но тоже есть вопросы.
у гитлаба своих проблем с интерфейсом хватает ))
источник

SZ

Sergey Z in rannts
Ну так можно и до написания красивого и эффективного кода через воспитательные беседы дойти :)
источник

in

ildar nizamov in rannts
Sergey Z
Ну так можно и до написания красивого и эффективного кода через воспитательные беседы дойти :)
пусть сразу пишут без ошибок
источник

AM

Artem Malyshev in rannts
Sergey Z
Ну так можно и до написания красивого и эффективного кода через воспитательные беседы дойти :)
Поведение в issues и PRs проверяется так же автоматически как и код. Например, через danger js.
источник

AM

Artem Malyshev in rannts
Нужен воспитательный линтер, а не беседы.
источник

RB

Roman Bolkhovitin in rannts
Человек укушенный Соболевым детектед )))
источник

SZ

Sergey Z in rannts
Воспитательные беседы - такое себе, в формате родитель-ребёнок они могут работать. В формате коллега-коллега они превращаются в обычный срач
источник

SZ

Sergey Z in rannts
Roman Bolkhovitin
Человек укушенный Соболевым детектед )))
Ты так говоришь, будто это что-то плохое :)
источник

💭П

💭 Руслан Прохоров... in rannts
Sergey Z
Ты так говоришь, будто это что-то плохое :)
Да. Поэтому, что нет единственно верного решения/подхода
источник

VR

Vasily Ryabov in rannts
Тогда надо договариваться о правилах с участием менеджера/тимлида/архитектора. Правда, порой могут и отмахнуться как от мелочи. Ну, тогда сами злобные буратины, просто повторное ревью не делаешь, письменно сославшись на отказ. Если это потом приводит к пропуску проблем, поднимаешь переписку. Впрочем, это уже аппаратная борьба. Не люблю это дело.
источник

RB

Roman Bolkhovitin in rannts
Sergey Z
Ты так говоришь, будто это что-то плохое :)
Да не, почему? Вполне имеющая право на жизнь точка зрения, просто есть ощущение, что иногда она возведена в абсолют и превращается в самоцель
источник

SZ

Sergey Z in rannts
Vasily Ryabov
Тогда надо договариваться о правилах с участием менеджера/тимлида/архитектора. Правда, порой могут и отмахнуться как от мелочи. Ну, тогда сами злобные буратины, просто повторное ревью не делаешь, письменно сославшись на отказ. Если это потом приводит к пропуску проблем, поднимаешь переписку. Впрочем, это уже аппаратная борьба. Не люблю это дело.
В аппаратной борьбе побеждает не здравый смысл а вес, потому заведомо не надо связываться, если веса нет
источник