Code Review

pilot

Смотрит ли ваш коллега по работе написанный вами код?
Пишете ли вы лучше, если знаете что коллега будет смотреть?
Нравится ли вам код коллег по работе? (В среднем)
Стыдно ли показывать им свой код?
Пишете ли вы docstrings в коде? wiki? readme?
Подписываете ли коммиты?

nikita270601

Смотрит ли ваш коллега по работе написанный вами код?
Смотрит.
Пишете ли вы лучше, если знаете что коллега будет смотреть?
Куда ж лучше-то?
Нравится ли вам код коллег по работе? (В среднем)
Ага.
Стыдно ли показывать им свой код?
Поначалу было немного, а потом как-то само прошло.
Пишете ли вы docstrings в коде? wiki? readme?
В основном, комментарии, а если что-то сильно запутанное, то какую-нибудь доку в wiki.
Подписываете ли коммиты?
«Подписываете» — в смысле, «сделал то-то, поправил такой-то баг»? Да.

pilot

> Смотрит.
По системе или когда использует?

nikita270601

По системе или когда использует?
Пока не посмотрит, код не попадёт в репозиторий.

pilot

Очень странная система
А из репозитория он смотреть не умеет?
Почему так устроено?

nikita270601

В частности, для того, чтобы весь код, который лежит в репозитории, был прочитан хотя бы двумя людьми.
Могу в приват дать ссылку на видеоролик, где Один Известный Человек рассказывает про этот процесс разработки и про инструмент (разработанный этим человеком) для поддержки такого процесса. Рассказывает довольно скучно, впрочем, но информативно.

Papazyan

>>Смотрит ли ваш коллега по работе написанный вами код?
Да.
>>Пишете ли вы лучше, если знаете что коллега будет смотреть?
Только для того, чтобы не д**сь и не пришлось переписывать.
>>Нравится ли вам код коллег по работе? (В среднем)
По барабану. Свой все равно лучше.
>>Стыдно ли показывать им свой код?
Нет, пусть учатся!
>>Пишете ли вы docstrings в коде? wiki? readme?
Да, на английском. Но это даже приятно, ведь кому-то придется это читать.

kruzer25

Смотрит ли ваш коллега по работе написанный вами код?
Ага...
Пишете ли вы лучше, если знаете что коллега будет смотреть?
Конечно
Стыдно ли показывать им свой код?
Какой?
Пишете ли вы docstrings в коде? wiki? readme?
Только комменты для иде, поясняющие всякую фигню, например, тип возвращаемого значения (в самом пхп это указание не поддерживается)
Подписываете ли коммиты?
Не понял, что имеется в виду под "подписываете"?

danilov

Не понял, что имеется в виду под "подписываете"?
Значит, не подписываешь
Каждый коммит сопровождается (или нет) кратким брифом внесённых изменений.
Про себя - стыдно на вопросы отвечать

kruzer25

Каждый коммит сопровождается (или нет) кратким брифом внесённых изменений.
Значит, подписываю
Оставить комментарий
Имя или ник:
Комментарий: