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