SY
какая у меня будет реакция ?
Блин, когда в проект лезет десяток человек, реакция на то, что твой чудесный код своими грязными лапами кто-то потрогал должна быть... Никакая :))
Size: a a a
2020 May 16
P
я закомитил пуллреквест что-то обсудили, его замерджили, потом на следующий день/на следующей неделе я листаю код и помню, что вот тут вот я по-другому делал и мне не нравится, как там поменяно - лезу в историю и вижу, что человек, который мне сказал, что код зашибись - лез и исправлял этот самый мой код и еще запушил это без PR
Если человек может пушить без ПР в мастер, то уже что-то не так идёт
A
Блин, когда в проект лезет десяток человек, реакция на то, что твой чудесный код своими грязными лапами кто-то потрогал должна быть... Никакая :))
Я описал другую ситуацию, зачем передергивать ?
SY
Я описал другую ситуацию, зачем передергивать ?
Ок, ок. Да и не бывает такого, что прям "код зашибись". Что-то можно и не углядеть и поправить потом. Это нормально)
A
это нормально, мне из текста показалось - что человек именно специально пойдет менять код ради того, что ему не понравилось, но в PR он это не упомянул
SY
это нормально, мне из текста показалось - что человек именно специально пойдет менять код ради того, что ему не понравилось, но в PR он это не упомянул
Я и не спорю) если это критикал, то не фидбечить - выстрел себе в ногу
БС
я закомитил пуллреквест что-то обсудили, его замерджили, потом на следующий день/на следующей неделе я листаю код и помню, что вот тут вот я по-другому делал и мне не нравится, как там поменяно - лезу в историю и вижу, что человек, который мне сказал, что код зашибись - лез и исправлял этот самый мой код и еще запушил это без PR
Мой друг Вадим жил в общаге с одним парнем в комнате. Готовили они по-очереди. Но Вадим любил резать картошку в суп покрупнее, а этот парень - совсем мельчил. Самое веселое было, когда Вадим сварит суп, уходит на занятия или еще куда. Приходит - а в супе вся картошка перерезана в труху :)
D𝔇
это нормально, мне из текста показалось - что человек именно специально пойдет менять код ради того, что ему не понравилось, но в PR он это не упомянул
Он сегодня кстати ещё одну статью на хабр вбросил - про любовь к визуальной красоте кода
D𝔇
Не сегодня, вчера)
https://m.habr.com/ru/post/502134/
https://m.habr.com/ru/post/502134/
БС
Он сегодня кстати ещё одну статью на хабр вбросил - про любовь к визуальной красоте кода
О, а я этого товарища уже читал несколько раз. Он по жизни депрессивный и наслаждается своими страданиями. А тут, значит, решил что и других мучить это нормально? Боже упаси от таких коллег!
A
Т.е. он таки откаментил. Если это не критично по функциональности, а код стайл какой-нибудь, то пофиг в целом. Если потенциальная утечка по памяти, то вряд ли бы он просто заапрувил
На мой взгляд в код ревью нужно проявлять уважение к другим. Даже если перед тобой человек, который только пришёл на работу/сменил деятельность и тд и тп. Писать замечания и делать код лучше - это нормально. Но важны детали, особенно если это начинает переходить в личное. Но другой вопрос, как на это реагировать)
A
О, а я этого товарища уже читал несколько раз. Он по жизни депрессивный и наслаждается своими страданиями. А тут, значит, решил что и других мучить это нормально? Боже упаси от таких коллег!
Потом представь, сменишь ты работу и встретишь его, а он это сейчас прочитал 😅
SY
О, а я этого товарища уже читал несколько раз. Он по жизни депрессивный и наслаждается своими страданиями. А тут, значит, решил что и других мучить это нормально? Боже упаси от таких коллег!
У всех свои загоны) главное, чтобы твои загоны более-менее уживались с загонами твоих коллег. Мб у него в команде все такие)
D𝔇
Потом представь, сменишь ты работу и встретишь его, а он это сейчас прочитал 😅
Он шарпист)
D𝔇
Блин, всегда пушу в мастер без ПР :-(
Не прогоняя локально тесты 😊
SY
Блин, всегда пушу в мастер без ПР :-(
😂 где-то видел статью про типа анти гитфлоу - разработку в одной мастер ветке. Аля свн