2019-09-12
Дистанциируемся от кода.
Программисты... они как-то очень трепетно относятся к своему коду. Это ж плод мысли, усилий интеллектуальных (и немножко yaml конфигов и копипасты со Stack Overflow).
Ясен хуй, что критика кода воспринимается как личный выпад. Отсюда все эти обиды и конфликты во время код-ревью. Мне в гуглоформу относительно регулярно пишут про "плохие" и "токсичные" ревью, про синдром вахтёра у коллег. Так что тема больная.
На самом деле я такой же, иногда читаю ревью и думаю "бля, ну вот ты мудак какой, нашёл до чего доебаться, а!". Потом, конечно, вспоминаю, что это просто код, который я написал так, как вышло в данных обстоятельствах. Да и писал я его не так как хотел, а так как получилось вклинить в существующую систему, с учетом сроков, эджкейсов и существующего дизайна и подходов, принятых в команде. Да и в целом, я хотел другую таску взять, но не успел. Короче, моего там не так уж и много, если подумать.
Есть у меня коллега, которая регулярно указывала мне на опечатки и грамматические ошибки в коде, в описании пулл-реквестов, в документации, в названиях бранчей и даже в коммит-мессаджах. Натурально, я одно время думал, что она меня ненавидит, и пытается усложнить мою жизнь. Трудно было не принимать это на свой счёт.
Ну и вместо того, чтобы жаловаться, эскалировать или начать огрызаться, я спросил: а собственно чего так? Почему ей это так важно? Сказала, что вот её аж трисёт когда ошибки видит, ещё со школьных времён, когда её дрючили за ошибки. Говорит, что теперь они "кричат" на неё, бросаются в глаза.
Понятно, что тут особо ничего не изменить, в итоге я поставил себе спеллчекер в IDE, и сказал, что то типа -- сорян, яж не нейтив спикер, но буду стараться.
Ошибки в коде/логах справлял, если указывала. А к другим временным штукам типа имён бранчей и ошибок в PR попросил не придираться, так как мне неприятно, а пользы от этого не много.
На чём-то таком и сошлись, а могли бы и поругаться, да?
К чему это всё? А к тому, что надо дистанциироваться от своего кода. Написал, запушил? Всё, теперь это общий код, а не твой. И претензии они не к тебе лично, а к системе в целом. Ну и к личным проблемам ревьюера, не без этого.