Как поступать с душными комментарии во время код-ревью?

Часто бывает что инженер, проверяющий твой код, оставляет какие-то бестолковые комментарии. Иногда они даже верны, но смещают фокус с основной задачи.

Раньше я вздыхал, и исправлял все замечания, что конечно замедляло время до закрытия задачи, и вообще бесило страшно.

Став старше, я пытался вступать в спор, рассказывая, что такие комментарии отнимают уйму времени, размазывают фокус и конечно же вносят больше проблем (редко кто тщательно тестирует код после внесения таких правок). Опять же такие споры затягивали ревью и изрядно трепали нервы.

Сейчас на многие замечания, если они не относятся к решению текущей задачи, я создаю тикеты в джире.

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

* Во-первых это обезоруживает комментатора: его замечания учтены, и вроде как неудобно заставлять тебя включать в PR другую задачу.
* Во-вторых, есть возможность привлечь команду к оценке важности этих изменений (когда тикет будут включать в спринт)
* И на последок, это просто ускоряет процесс, и наглядно показывает цену таким "мелким" замечаниям и предложениям.

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

+ 100500 Бывало такое, что на ревью кода получаешь 17 комментариев и это выливается в неделю работу + новые задачи + расфокус

только рожая трёх детей одновременно женщина может понять страдания девопса, когда ему приходится программировать на перле вместо баша

Chrome: Settings -> Advanced -> Site Settings -> Notifications -> change "Ask Before Sending" to "Blocked".

Все, теперь не будет бесконечных поп-апов "можно ли спамить вас уведомлениями".

You're welcome.

Такой вчера сюрприз получил от Vodafone: сломался инет основной, ну я ж расшарил точку доступа на телефоне (а там анлим от водафон) и через 5 минут инет отвалился и смс от водафона типа “раздача инета с вашего телефона недоступна и подключите услугу за 50 грн в месяц” я немного подофигел, что не могу расшарить инет с телефона и что водафон блокирует такое и заставляет покупать допуслуги для этого.
Однако мне ж стало интересно как они детектят такие вещи и блокируют, полез читать и оказалось что все примитивно - TTL. Вообщем, поменял на маке TTL с 64 на 65 и все ок. А оператор смотрит на TTL, если он проходит через телефон (то есть роутер) то ttl уменьшиться на 1. То есть на конечном устройстве надо его поднять на 1 и все будет ок.

Команда для мака
sysctl -w net.inet.ip.ttl=65

Для linux

sysctl -w net.ipv4.ip_default_ttl=65

водафон вотафак

Классная статья про использование терраформ https://medium.com/driven-by-code/the-terrors-and-joys-of-terraform-88bbd1aa4359

Топология Звезда - центр лёг, всему пизда

как раз недавно картинка была в тему

Тесты на проде

Напоминаю, что через 26 дней начнется Хактоберфест https://hacktoberfest.digitalocean.com/

В прошлый раз зафигачил 5 коммитов в паблик репы и получил футболку и стикеры)