Инструкция как заставить команду плакать или как работает авторитет
Однажды я случайно удалила спецификацию OpenAPI всего продукта и это попало 🥁на прод 🥁
✏️Инструкция:✏️ удалить спеку, проверить третьем глазом, залить это в гит и отправить на ревью лиду бэков в пятницу вечером, как же без этого? А если добавить что вам срочно в отпуск, то: лид апрувит и смерджит всё в мастер. Вы счастливые уходите в отпуск. В понедельник QA, конечно, вашего счастья разделять не будут.
Как так вышло?
Во-первых, сыграл мой авторитет. Я на тот момент лучше всех понимала наше API и как раз настраивала процесс работы с ним и сваггером. После моего пул реквеста лид что-то заподозрил и пришёл ко мне с вопросом «Что за фигня, Катя?». Но я с полной уверенность «Тоже видела, как-то странно отображается, да, но я всё проверила, и там всё хорошо» убедила его не копать дальше. Ну а зачем копать глубже, если я вроде бы одна из немногих, кто вообще в этом шарит, и единственная, кто отвечает?
Во-вторых, чтобы процесс ревью работал, нужно, чтобы ревьюер очень чётко понимал, что от него ждут. В нашем случае цель была — отсеять новичков. Те меня процесс ревью как то не учитывал.
Очень неудачная спека на 30 тысяч строк. В гитоподобных системах в процессе ревью с таким тяжело работать.
Я собиралась в отпуск, было срочно. Поэтому сама не стала вникать, а почему всё светится красным? И коллега тоже не хотел меня подставлять перед отпуском (подставил в итоге себя).
Так, на два дня, мой авторитет, неудачный YAML и неотлаженные процессы затормозили разработку. После этого случая я стала замечать, что опытные тоже ошибаются, часто (Вот здесь можно почитать о другом фейле), но из-за авторитета и возможно лени, это чаще пропускают. Только вот задачи такие специалисты делают более важные, которые чаще могут что-то сломать.
В вашей практике как? Часто с таким сталкиваетесь? Что было самым фатальным?
Post #416
352