The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

В GitLab 10.7 открыт код Web IDE, opennews (?), 23-Апр-18, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


43. "В GitLab 10.7 открыт код Web IDE"  –2 +/
Сообщение от anonymous (??), 23-Апр-18, 17:52 
как сейчас можно делать код ревью? в gitlab что-то есть для этого?
Ответить | Правка | Наверх | Cообщить модератору

45. "В GitLab 10.7 открыт код Web IDE"  +/
Сообщение от Аноним (-), 23-Апр-18, 18:30 
можно. merge requests там есть с контекстными комментариями, и судя по новости, туда же прикрутили редактор кода наконец, для мини-правок. Мы пользуемся для код ревью им, человек около 15, Сишный проект.
Ответить | Правка | Наверх | Cообщить модератору

49. "В GitLab 10.7 открыт код Web IDE"  +/
Сообщение от Аноним (-), 23-Апр-18, 18:36 
подробнее:

MR без аппрува не мержутся в мастер. В обязанности разрабов входит регулярный мониторинг и ревью текущих MRs, у которых недостаточно условий для аппрува. Условиями аппрува являются 2 thumbs up у MR. Новых членов команды обучаем (обычно просто вводим в курс дела и первое время задаем вектор, как и что делать). В обсуждение MR часто кастуем нужных чуваков, кто имеел дело с подсистемой и связан с изменениями, ждем их thumbs up.

Сейчас в очереди перманентно около 100 MR, в день аппрувятся около 10. Размер MR совершенно разный, никак не оговорено. Кодстайл, конечно, достаточно формализован, хрень в описании к коммитам не пройдет и всё такое.

Ответить | Правка | Наверх | Cообщить модератору

46. "В GitLab 10.7 открыт код Web IDE"  +/
Сообщение от Аноним (-), 23-Апр-18, 18:30 
Есть ревью Merge Request'ов.  Если не вливать изменения в мастер-ветку без проверок, то схема подходит.

Также есть комментирование изменений в коммитах. Но это менее удобно.

Ответить | Правка | К родителю #43 | Наверх | Cообщить модератору

47. "В GitLab 10.7 открыт код Web IDE"  +/
Сообщение от Аноним (-), 23-Апр-18, 18:31 
Да. Можешь посмотреть, как это выглядит, например, здесь: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/18532...
Ответить | Правка | К родителю #43 | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру