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