В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

codingteam

72 membersпожаловаться на группу
2020 June 25

Dv

Dr. Friedrich von Ne... in codingteam
codingteam@cjr
Minoru
а как вы думаете, кто должен отмечать ревью-комменты на GitHub как «resolved»? Я вот считаю, что это должен быть автор комментария: он нашёл проблему, и ему решать, пофикшена она или нет
Да, я тоже так считаю. Ревьюер отметил проблему, автор её исправляет, ревьюер смотрит ещё раз и помечает, что она исправлена.
источник
11:05пожаловаться#1

Dv

Dr. Friedrich von Ne... in codingteam
Жаль, что GitHub не даёт инструментов, чтобы я мог пометить именно такой workflow как желательный.
источник
11:06пожаловаться#2

AO

Alibek a1batross Oma... in codingteam
Система ревью на гитхабе вообще странная.
источник
11:06пожаловаться#3

c

codingteam@cjr in codingteam
Minoru
да, в связи с этим и вопрос. У меня некоторые люди сами отмечают. Некоторые даже игнорят просьбы так не делать!
источник
11:06пожаловаться#4

AO

Alibek a1batross Oma... in codingteam
Там есть две кнопочки. Оставить комментарий и начать ревью.
источник
11:06пожаловаться#5

AO

Alibek a1batross Oma... in codingteam
Во втором случае твой комментарий девается в какую-то жепу.
источник
11:06пожаловаться#6

Dv

Dr. Friedrich von Ne... in codingteam
У нас в Space (это наш аналог приватного GitHub) есть специальный режим, "turn-based review", который формализует такой подход. Я долго плакался в чатике, чтобы чуваки его включили, и теперь я доволен.
источник
11:07пожаловаться#7

c

codingteam@cjr in codingteam
Minoru
@a1batross: ну, на фоне Gerrit очень хорошая, я считаю :) Другие не юзал
источник
11:07пожаловаться#8

Dv

Dr. Friedrich von Ne... in codingteam
Там всегда понятно, чей ход, и какие проблемы/обсуждения остались открытыми.
источник
11:07пожаловаться#9

AO

Alibek a1batross Oma... in codingteam
И автору патча не отправляется уведомление. Я так от одного ждал фиксов, а в итоге было проще лично спросить.
источник
11:07пожаловаться#10

AO

Alibek a1batross Oma... in codingteam
Теперь жму просто Add single comment.
источник
11:07пожаловаться#11

c

codingteam@cjr in codingteam
Minoru
Gerrit, впрочем. я тоже юзал очень мало. Но хоть чуток. И там ногу сломить можно в этих их патчсетах
источник
11:07пожаловаться#12

Dv

Dr. Friedrich von Ne... in codingteam
Alibek a1batross Omarov
И автору патча не отправляется уведомление. Я так от одного ждал фиксов, а в итоге было проще лично спросить.
Ничего не отправляется, пока ты не нажмёшь "Finish review".
источник
11:07пожаловаться#13

c

codingteam@cjr in codingteam
Minoru
да, после отправки ревью всё отправляется исправно. Я неоднократно получал письма, когда меня ревьювили
источник
11:08пожаловаться#14

Dv

Dr. Friedrich von Ne... in codingteam
Это на самом деле хорошая фича. Ты читаешь код, пишешь замечания, потом дочитываешь до какого-то важного места и можешь их поправить, переделать. И в конце всё батчем отправляешь. Это пришлёт уведомление.
источник
11:08пожаловаться#15

AO

Alibek a1batross Oma... in codingteam
Как resolved можно считать как что-то изменилось. Обычно все понимают всё с одного раза и так исправляют.
источник
11:08пожаловаться#16

c

codingteam@cjr in codingteam
Minoru
я обычно пишу по ходу всякую недоформулированную фигню, а потом удаляю слишком «низкоуровневые» комменты, если нахожу какую-то большую проблему
источник
11:09пожаловаться#17

Dv

Dr. Friedrich von Ne... in codingteam
Alibek a1batross Omarov
Как resolved можно считать как что-то изменилось. Обычно все понимают всё с одного раза и так исправляют.
Ну, такой подход тоже сработает, я думаю :)
источник
11:09пожаловаться#18

FO

FORTRAN ONE LOVE in codingteam
gsomix
Dr. Fortran von Never
О. Идеально.
источник
11:09пожаловаться#19

AO

Alibek a1batross Oma... in codingteam
Dr. Friedrich von Never
Ну, такой подход тоже сработает, я думаю :)
ну он и работает
источник
11:09пожаловаться#20