Skip to content

При пустом дифе выдержка для ревьюера снова начинается с начала файла #1124

Description

@gHashTag

Часть #1090. Поправка к моему же закрытию #1123 — я закрыл его по одному прогону и переоценил починку.

Доказано двумя прогонами одной задачи

Повтор ревью #1117, граница rings/SR-02/ChatViewModel.swift, критерии те же.

Прогон, где пчела меняла файл: ревьюер цитирует саму реализацию, parsed=2 recorded=2.

Прогон, где пчела не меняла ничего:

The diff reports "(no changes detected)", and the provided file is
truncated at 500 of 5462 lines — so I can see the data structures but
not the code that uses them.

parsed=0.

Причина

Выдержка следует за изменёнными местами. При пустом дифе якоря нет, и выборка откатывается к первым 500 строкам — ровно к тому поведению, которое #1123 должна была устранить. А повторное ревью готовой работы это всегда пустой диф; именно там обрезка и вредит.

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

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

Границы

rings/SR-02/ChatViewModel.swift

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