Skip to content

PR пчелы несёт 2010 файлов вместо одного: база выбрана не та #1135

Description

@gHashTag

Часть #1090. Последствие моей же починки #1110.

Доказано прогоном

Задача #1134 прошла весь цикл впервые с #1102: пчела закоммитила один файл, ревьюер дал вердикт, Королева приняла и открыла PR. И вот что она открыла:

state=OPEN  files=2010  base=dev  head=queen/1134-write-docs-seven-links-md-describing-the

Две тысячи десять файлов вместо одного.

Причина

В #1110 я заставил её брать базой ветку по умолчанию удалённого репозитория — чтобы она перестала сливать в ту ветку, где работает человек. Но ветка пчелы отрезана от рабочей ветки, а та ушла от dev на целую ночь коммитов. Разница между веткой пчелы и dev — это вся ночь, а не работа пчелы.

Обе прежние крайности неверны: текущая ветка как база даёт слияния в чужую работу, ветка по умолчанию даёт PR со всем подряд.

Готово, когда

  • PR пчелы содержит только то, что написала пчела: базой служит точка, от которой отрезана её ветка.
  • Слияние по-прежнему не идёт в ветку, где работает человек, без его участия.
  • Доказано прогоном: PR для задачи с одним файлом содержит один файл.
  • Проверка ломается, если вернуть базой ветку по умолчанию.

Границы

rings/SR-02/QueenBranchCommitter.swift и rings/SR-02/ChatViewModel.swift

No activity

Activity on this issue will appear here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions