Твой коммит — твоя ответственность 🫡

Иногда, чтобы стать лучше как разработчик — достаточно внедрить одну простую привычку: всегда проверяй то, что ты коммитишь. Это действие, которое помогает избежать не только многих технических проблем, но и недоверия со стороны коллег.

Кажется, что это мелочь? Тогда давайте посмотрим на классические примеры, которые объединяет одно — торопливость, безответственность и невнимательность. 😰

Топ отговорок при пуше "сломанного" коммита:

🌷 "Я просто перенёс этот кусок кода в другой файл, что могло пойти не так?"

Оу, на самом деле — очень многое: забыли про импорты, сбили зависимости, не учли изменения логики. Вы подумали, что это всего лишь косметика, но эта косметика сделала из вас ненадёжного разработчика в глазах коллег буквально в пару строк..

🌷 "Я изменил код по-мелочи."

Добавили условие, сделали проверку чуть строже, заменили одну строку. На первый взгляд, что тут проверять? Но позже приходит сообщение: "Почему кнопка отправки заявки перестала работать? Мы теряем клиентов и деньги уже на протяжении 5ти часов..". И хорошо, если это сообщение вам пришло не в 2 часа ночи, а в рабочее время.

🌷 "Я просто переключил API, мне Петька сказал, что там всё супер."

Перенесли ответственность на другого человека.. Поменяли адрес для API-запросов, уверенные, что ничего не изменится. Но структура данных изменилась, API отвечает с ошибкой, а приложение просто не знает, как обработать новый формат. Итог: пользователи все в ошибках и шоке, а вы — с горящей пятой точкой и чувством вины бежите править свой код. 😰

🌷 "Конфликты были легко разрешены, тесты при мердже пройдены, значит всё супер."

Не делайте так. Довольно часто тесты покрывают лишь критические части приложения и если вы не сломали что-то критическое, то это не гарантирует того, что вы не сломали что-нибудь ещё - то, что тестами покрыто не было.

🌷 "Копирование чужих решений."

Нашли на форуме «идеальный» кусок кода, вставили в проект, запушили и успокоились. Однако в нём были использованы переменные, которых у вас нет, или отсутствует логика, необходимая для корректной работы, или, опять же, нет нужных зависимостей. Вы не протестировали подумав: «я глазами пробежался, там вроде всё ок, даже тестить не нужно», и вот уже к вам, спотыкаясь, бежит недоброжелательный менеджер, чтобы сообщить размер будущего штрафа.

Почему это важно?

Казалось бы, что тут такого: ну ошибся, откатили, поправили и дальше работаем. Но регулярное игнорирование проверки своих изменений — это привычка, которая подрывает качество вашей работы. Когда вы не проверяете, что вливаете — вы показываете, что не относитесь ответственно даже к своим же изменениям, не говоря уже о проверке изменений своих коллег, которая становится обязательной при карьерном росте.

Настоящий профессионал всегда осознаёт, что его код — это его ответственность. Даже если вы скопировали решение с форума, взяли фрагмент из прошлой задачи или применили чью-то подсказку — в финале ваша задача проверить, что всё это работает корректно! 👮‍♀️

Проверяйте, тестируйте и будьте уверены в том, что ваш коммит делает проект лучше, а не наоборот.

P.S. Все примеры и отговорки вымышлены. Любые совпадения случайны. 😉