> ## Content Index
> Fetch the complete content index at: https://www.rozhkov.me/llms.txt
> Use this file to discover other available public pages before exploring further.

# „Подивись плз мердж реквест“
- URL: https://www.rozhkov.me/please-review-my-merge-request/
- Published: 2023-02-01T08:00:00.000Z
- Updated: 2023-02-01T08:14:17.000Z
- Author: Vova Rozhkov
- Tags: робота

— пишу я колезі у слак.

— „Можеш дати контекст?“, — відповідає той.

Я зазвичай прискіпливо ставлюся до [коміт меседжів](https://www.rozhkov.me/why-comment-commits/) та всього де є букви й намагаюсь спростити рев'юеру роботу, тому мій MR вже містить всі необхідні пояснення. Подумки зітхаю, копіюю текст звідти у слак.

Через годинку приходить на пошту нотифікація — рев'юер написав що можна додати перевірку на `null`. Перевірки там не треба, бо метод приватний та викликається лише з місця де йому передаються ключі хешмапи, які не можуть бути `null`. Пишу це у коменті до коменту.

Проходить кілька днів, MR стоїть, відповіді немає. Треба знову пінгувати людину в слак.

Задумався про це. Як могло б бути краще? Я вже давно не рев'юив ніяких мердж реквестів, тому не пам'ятаю чи поводив би я себе так само. Але зазвичай раз на годину я дивлюся в пошту та опрацьовую вхідні, тому задача типу такої в мене б довго не протрималась.

Проте це я, і вимагати від інших такої ж відповідальності, якщо це не зафіксовано у корпоративних політиках («SLA на рев'ю мердж реквеста має складати 1 день»), нерозумно.

![](https://images.unsplash.com/photo-1523240795612-9a054b0db644?crop=entropy&cs=tinysrgb&fit=max&fm=jpg&ixid=MnwxMTc3M3wwfDF8c2VhcmNofDQxfHxjaGF0fGVufDB8fHx8MTY3NTA3MjY2Ng&ixlib=rb-4.0.3&q=80&w=2000)

Photo by [Priscilla Du Preez](https://unsplash.com/@priscilladupreez?utm%5Fsource=ghost&utm%5Fmedium=referral&utm%5Fcampaign=api-credit) / [Unsplash](https://unsplash.com/?utm%5Fsource=ghost&utm%5Fmedium=referral&utm%5Fcampaign=api-credit)

Можна подумати, що слак все псує. Бо частина комунікації йде туди, частина в пошту, частина в гітлаб. Давайте [заберемо слак](https://www.rozhkov.me/why-async-remote-not-working/) і все буде добре! Можливо стало б краще, але гадаю що люди б просто знайшли інші канали комунікації. На моїй першій роботі з керівництво не запровадило корпоративний месенджер. Тоді ще не було слаків та діскордів, у світі панував IRC, але в нас не було нічого, тому працівники просто спілкувалися спочатку в ICQ, а потім в Skype. СекОпси прийшли б в жах!

Якщо ж встановити SLA, зобов'язати людей перевіряти MR швидше, то вони швидко почнуть LGTMити на все підряд.

Не знаю як правильно. Якби я був головним, то я б не брав людей, котрі не підходять мені культурно, намагався б якось визначити на співбесіді, кандидат надає перевагу синхронній комунікації чи асинхронній.

А тих, хто два дні поспіль не відповідає на комент, просто звільняв. Does not meet expectations.