В конечном счете, «что делать при проверке» зависит от вашего конкретного рабочего процесса / команды.
Если вы хотите, чтобы разработчики тестировали код как часть процесса, тогда они могут свернуть ветку с исходным кодом и использоватьчтобы запустить код для тестирования.
Если вы хотите, чтобы разработчики редактировали код как часть рецензирования, то у меня нет предложений, как управлять этим.Я думаю, что это просто не имеет смысла (кто рецензирует рецензента?), Но я привел свои рассуждения ниже.
Как правило, идея с запросом на извлечение заключается в том, что другие разработчики будут рассматривать измененияи предлагать улучшения обратно первоначальному автору, а не просто вносить изменения самостоятельно.Опять же, это специфический для рабочего процесса / коллектива вопрос, но я лично считаю очень грубым для другого разработчика просто редактировать мой PR, не разговаривая со мной в первую очередь.
Предполагается, что человек, который написал оригинальный кодобладает самыми современными знаниями о предмете / бизнес-требованиях / технических ограничениях, и поэтому предложения (и их обоснование) должны быть оценены ими.Опять же, это вещь, связанная с рабочим процессом / командой.