From dab48e9c0490d85dcb2e895e8e0e7fc67fb375a7 Mon Sep 17 00:00:00 2001 From: katemurao19 <106687261+katemurao19@users.noreply.github.com> Date: Wed, 28 Dec 2022 11:43:56 +0300 Subject: [PATCH] Update neighborhood-agreement.md --- neighborhood-agreement.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/neighborhood-agreement.md b/neighborhood-agreement.md index ed83f4b..c28625c 100644 --- a/neighborhood-agreement.md +++ b/neighborhood-agreement.md @@ -19,7 +19,7 @@ Правило бойскаута | Оставь код репозитория лучше, чем он был до твоего прихода. Куда хочешь, туда и пушь | Делать PR можно в любой репозиторий, если это не запрещено внутренними политиками безопасности или соответствиями внешним стандартам. В чужой код со своим уставом не лезь | У команд есть Coding Standard и политика Code Review. Нужно использовать Coding Standard команды, которая владеет кодом. Если Code Standard в команде отсутствует — подсвети проблему оунеру. -Запушил в чужой код, не мержь без согласия владельца | Следуем политике Code Review команды, которая владеет кодом. +Запушил в чужой код, не мёржь без согласия владельца | Следуем политике Code Review команды, которая владеет кодом. Написал новый код, напиши и тесты | Тесты качественные и являются частью документации. Написал новый код, а тесты не упали? Проверь и исправь тесты | Покрываем тестами так, что баг ломает тесты. Если логику поломали, а тесты «зеленые», значит их надо исправлять. Исправляет CODE OWNER, который владеет этим функционалом. Кто сломал, тот и чинит | Кто сломал автотесты, уронил метрики или удалил логирование, тот их и исправляет.