В этом документе содержатся инструкции по проверке изменений. При рассмотрении изменений следует учитывать следующие критерии:
- Соответствует ли это изменение заявленной цели данного проекта?
- Допустимо ли внесение этих изменений в рамках существующей архитектуры проекта?
- Вносит ли это изменение конструктивные недостатки, которые могут вызвать проблемы в будущем?
- Соответствует ли это изменение передовым практикам, установленным для данного проекта?
- Является ли это изменение оптимальным способом выполнения описанной функции?
- Влечёт ли это изменение за собой какие-либо риски для безопасности или нестабильности, а также может ли оно нарушить работу существующих приложений?
Дополнительную информацию о проверке изменений см. в разделе «Проверка изменений» документации Gerrit.
Если вас назначили рецензентом изменений, вы должны получить электронное письмо со ссылкой на эти изменения. Чтобы проверить и отредактировать изменения, выполните следующие действия:
Щелкните ссылку в электронном письме, ведущую к изменению. Отобразится экран «Информация об изменении» в Gerrit. На этом экране представлено описание изменения, указаны автор изменения и все рецензенты, а также список файлов, связанных с изменением.
Ознакомьтесь с описанием изменений, чтобы понять их цель.
Щелкните файл, чтобы просмотреть внесенные в него изменения. Файл откроется в двухпанельном сравнительном окне: слева — существующий файл, справа — файл с предлагаемыми изменениями. Для просмотра файла:
С помощью мыши выделите (щелкните и перетащите) тот участок кода, к которому у вас есть комментарий или вопрос. Появится сообщение «Нажмите c, чтобы оставить комментарий».
Нажмите клавишу c . Появится поле для черновика комментариев.
Введите свой комментарий или вопрос и нажмите СОХРАНИТЬ .
Чтобы вернуться к экрану с информацией об изменении, нажмите на уникальный номер изменения (в левом верхнем углу экрана).
Продолжайте проверку каждого файла в рамках изменений.
После того, как вы просмотрите все файлы в изменении, вернитесь на экран «Информация об изменении» и нажмите «ОТВЕТИТЬ» . Появится экран «Отправить голоса по требованию».
Выберите один из следующих показателей для оценки изменений:
- +1 Мне нравится, но кто-то ещё должен это одобрить.
0 Нет оценки
-1 Я бы предпочёл, чтобы это не было отправлено в таком виде, здесь есть что исправить.
-2 Это изменение не следует отправлять
Нажмите «ОТВЕТИТЬ» , чтобы отправить свои комментарии и оценку разработчику изменений.
Если есть проблемы, требующие исправления, разработчик их устраняет и просит вас проверить исправления. В противном случае, проверка изменений считается завершенной.