Различия
Здесь показаны различия между двумя версиями данной страницы.
| Предыдущая версия справа и слева Предыдущая версия Следующая версия | Предыдущая версия | ||
|
abills:docs:development:git:ru [2016/03/23 09:31] anton |
abills:docs:development:git:ru [2017/11/01 11:33] (текущий) anton [Алиасы] |
||
|---|---|---|---|
| Строка 34: | Строка 34: | ||
| ====Рабочий процесс==== | ====Рабочий процесс==== | ||
| - | Создаём новую ветку для задачи\\ | + | Создаём новую ветку для задачи ''Msgs''\\ |
| + | # git checkout -b Msgs | ||
| В новой ветке:\\ | В новой ветке:\\ | ||
| Повторить нужное количество раз:\\ | Повторить нужное количество раз:\\ | ||
| Строка 44: | Строка 45: | ||
| </эти действия>\\ | </эти действия>\\ | ||
| - | Заливаем в основной репозиторий в ветку Msgs | + | Заливаем в основной репозиторий в ветку ''Msgs'' |
| # git push origin Msgs | # git push origin Msgs | ||
| Строка 130: | Строка 131: | ||
| </code> | </code> | ||
| + | ==== Процесс Code review для исполнителя ==== | ||
| + | ** Подготовка кода к review **\\ | ||
| + | - Убедиться, что функционал работает \\ | ||
| + | * Если это подключаемый функционал (через права или опцию в конфиге), убедиться что при выключении функционала система работает так же, как и до внесения изменений. | ||
| + | - Просмотреть код и убрать всё, что использовалось при отладке (_bp, Dumper, импорт модулей с этими функциями) | ||
| + | - Для алгоритмически сложных частей написать в комментариях, что именно делает код и почему использовалось именно это решение. | ||
| + | ** Залить код для проверки **\\ | ||
| + | - Залить в отдельную ветку для review (''текущая_ветка''_review) | ||
| + | - Слить основную ветку (dev или ветку review'ера) | ||
| + | - Поменять статус задачи на "На тестировании". Написать имя ветки в комментариях к задаче. | ||
| + | - Уведомить ответственного за review гарантированным методом (личный контакт или по договорённости) | ||
| + | ** Обработать результат review **\\ | ||
| + | После получения ответа \\ | ||
| + | Если задача перемещена в статус "Обратная связь"\\ | ||
| + | * Слить (merge) ветку ''_review'' в ветку задачи. | ||
| + | * В комментариях задачи просмотреть замечания (если есть) | ||
| + | |||
| + | Если задача перемещена в статус "Решена"\\ | ||
| + | * Ожидать решения ПМ по закрытию задачи. | ||
| + | |||
| + | ==== Процесс Code review для проверяющего ==== | ||
| + | - Перейти на ветку для проверки задачи | ||
| + | - Проверить выполнение | ||
| + | - Проверить функционал | ||
| + | - Проверить код (отметить подозрительные места, если есть). Где надо, выставить пометки ''# TODO: comments''. | ||
| + | - Залить свою ветку в ветку ''_review'' | ||
| + | - Оставить замечания в комментариях к задаче. | ||
| + | - Выставить нужный статус (''Решена'' или ''Обратная связь'' ) | ||
| ====Полезные статьи:==== | ====Полезные статьи:==== | ||
| http://asmodeus.com.ua/lib/category/programming/git/ | http://asmodeus.com.ua/lib/category/programming/git/ | ||