Mirror/dev-roadmap-avito-playbook
1
0
Fork 0
mirror of https://github.com/avito-tech/playbook.git synced 2026-03-14 06:02:56 +03:00
Code Issues Packages Projects Releases Wiki Activity

Update neighborhood-agreement.md

Browse Source
This commit is contained in:
katemurao19
2022-12-28 11:43:56 +03:00
committed by GitHub
parent 19a23783d1
commit dab48e9c04
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
neighborhood-agreement.md
View File

@@ -19,7 +19,7 @@
Правило бойскаута | Оставь код репозитория лучше, чем он был до твоего прихода.
Куда хочешь, туда и пушь | Делать PR можно в любой репозиторий, если это не запрещено внутренними политиками безопасности или соответствиями внешним стандартам.
В чужой код со своим уставом не лезь | У команд есть Coding Standard и политика Code Review. Нужно использовать Coding Standard команды, которая владеет кодом. Если Code Standard в команде отсутствует — подсвети проблему оунеру.
Запушил в чужой код, не мержь без согласия владельца | Следуем политике Code Review команды, которая владеет кодом.
Запушил в чужой код, не мёржь без согласия владельца | Следуем политике Code Review команды, которая владеет кодом.
Написал новый код, напиши и тесты | Тесты качественные и являются частью документации.
Написал новый код, а тесты не упали? Проверь и исправь тесты | Покрываем тестами так, что баг ломает тесты. Если логику поломали, а тесты «зеленые», значит их надо исправлять. Исправляет CODE OWNER, который владеет этим функционалом.
Кто сломал, тот и чинит | Кто сломал автотесты, уронил метрики или удалил логирование, тот их и исправляет.