arezvov 1 мар 2020 в 23:34

Ревью кода системы средствами git

3 мин

18K

Git *GitHub *Управление разработкой *

+14

Комментарии 7

unabashed 1 мар 2020 в 23:55

Спасибо за статью. Метод практически легко реализуемый и удобный, но вставлю все же свои 5 копеек: документ, а также любая таск- или crm- системы на практике удобнее для менеджеров, которые не используют в своей работе системы контроля версий. Я, все же, в том лагере, который предложит оставить привычные инструменты. Git — разработчикам, jira — управленцам. Но Ваш способ хорош.

Sdin 2 мар 2020 в 08:29

Думаю автор как раз имел в виду: Git — разработчикам, т.к. изначальное требование: ревью кода. Здесь могут быть выполнены замечания как по архитектуре в целом (как можно сделать то же самое проще/более гибко/быстрее/производительней, замечания к API и т.п.) и по таким «мелочам» как стилистика кода (code style). Данная информация поможет качественно оценить код старшим программистам на принимающей стороне, либо даже самому ревьюверу, чтобы в последующем сделать анализ и выдать «документ» об общем состоянии кода для «менеджера» с доказательной базой (та самая ветка с комментариями).

arezvov 2 мар 2020 в 20:55

Даже нечего добавить, спасибо!

arezvov 2 мар 2020 в 20:54

Спасибо за отзыв! Я предложил метод, а использовать или нет — ваше дело.

unabashed 2 мар 2020 в 20:56

Моё. Обязательно буду использовать там, где возможно :) Спасибо!

BekoBou 3 мар 2020 в 01:43

Я не понимаю зачем создавать ветку? Почему не сразу в Issue писать со ссылками на коммит?

https://github.com/oktend/system-review-example/commit/ed8639e5aacf7d90935a9dc72cc8e085317d6610#diff-7a9076d6d94e62c13d641aa71f19ae8eR2 — definetly code needs some comments here

По сути получится точно как у вас в примере с PR, только без плясок с ветками. И задачи будут именно там, где им место.

arezvov 3 мар 2020 в 21:40

Получится не так, в описанном подходе комментарии оставляются при желании в самом исходном коде, а в интерфейсе гитхаба или аналогичной системы можно продолжить обсуждение.
Ну и ревью — это далеко не всегда задачи, это могут быть вопросы, либо указания на то, что сделано хорошо, или еще что-то.

Ваше непонимание говорит о том, что потребности в таком подходе у вас еще не родилось, наверное не было таких задач )

Зарегистрируйтесь на Хабре , чтобы оставить комментарий

Ревью кода системы средствами git

Комментарии 7

Публикации

Истории