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