[devel] APT development /code review tool

Anton Farygin rider на basealt.ru
Пт Окт 25 15:46:25 MSK 2019


On 25.10.2019 15:41, Andrey Savchenko wrote:
> On Fri, 25 Oct 2019 15:32:08 +0300 Anton Farygin wrote:
>> On 25.10.2019 15:30, Andrey Savchenko wrote:
>>> On Fri, 25 Oct 2019 08:03:51 +0300 Anton Farygin wrote:
>>>> On 25.10.2019 0:44, Michael Shigorin wrote:
>>>>> Поэтому если желающие развернут тот же gerrit для добровольного
>>>>> участия в рецензировании кода -- в добрый путь.
>>>> у желающих давно уже развёрнуто. Множество различных по величине
>>>> проектов активно используют те или иные инструменты для online code review.
>>>>
>>>> Собственно сам процесс code review (наравне с CI) часто является одним
>>>> из этапов для прохождения любых патчей в проект.
>>>>
>>>> Попробуй ради интереса сделать патчик к FreeIPA, LibreOffice, KDE, MySQL
>>>> или QT  и отправить его в апстрим.
>>> Действительно, попробуй что отправить через gerrit и т.п. code
>>> review в ядро или glibc. О результатах напиши.
>>>
>> Через gerrit вообще невозможно что-то отправить. У него другой workflow.
> Давай буквоедством не заниматься. Очевидно, что имелось ввиду
> "засабмитить свой патч в апстрим как конечный результат обсуждения
> с апстримом и возможной модификации предложенного патча по
> результатам данного обсуждения". Если ты затребуешь gerrit для code
> review твоих изменений ядра со словами "нихацу использовать
> e-mail", тебя пошлют далеко и надолго, или просто проигнорят.
>
Естественно. Но у нас согласись немного другая ситуация. А именно - 
отсутствие внятных договорённостей между участниками разработки о 
средствах и инструментах этой разработки.

Т.е. - если бы у нас было бы утверждёное policy, регламентирующие 
процесс взаимодействия по code review между участниками разработки, то 
данный вопрос даже бы не поднимался.



Подробная информация о списке рассылки Devel