Записи, которые прошли через Дифференциальное отображение в системе аудита Фабрикатора

Question

Довольно простой вопрос, вот в чем проблема.

Наш рабочий процесс: вы выполняете свою работу в своей местной ветви, совершаете часто и подробно и хотите выполнять разную работу.

• дуга 'Происхождение/superAwesomeBranch' DIFF

(superAwesomeBranch наша главная ветвь развития. Я не знаю, почему мы не используем, это просто, как это)

разница доходит до Phabricator и обратная связь потоков мы делаем следующее:

• мерзавца тянуть --rebase (для хорошей меры, мы уверены, часто перебазироваться)

• Внесите свои изменения/новые обязательства.

• дуговых диффы --update Dxx

• коммита, будет принято ...

• дуга земли --onto superAwesomeBranch --merge (мы хотим сохранить нашу историю совершить, я знаю, что мы могли бы просто установите это в файле .arcconfig)

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

Является ли наш рабочий поток испорченным, не хватает ли шага/флага команд?

Answer 1

Фабрикатор не имеет дело с этим случаем очень интуитивно. По крайней мере, в некоторых случаях рассматривается только фактическое слияние.

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

Обходным путем было бы сбросить флаг --merge, но, очевидно, вы потеряете историю. This article объясняет, почему мы делаем это в наших рабочих процессах и почему это значение по умолчанию, но не все считают, что это привлекательно, и нам следует лучше поддерживать рабочий процесс слияния.

Исторически сложилось так, что все рабочие процессы Phabricator на Facebook были «один отзыв = один фиксация». В дикой природе многие общие рабочие процессы - «один отзыв = одна ветвь». Мы постепенно улучшали обработку этого, но он все еще отстает от рабочих процессов «one commit».