Телеграмм чат группы rannts страница 3423

А вообще странно, тикет на рефакторинг был или ты а решение проблемы?

18:36пожаловаться #1

БС

Байт Словович in rannts

Sergey Z

а скажите мне, имеет ли хоть какой-либо смысл мержить рефакторинг, если изначально он затевался с целью решить конкретную проблему, но по окончании выясняется, что изначальная проблема не решена от слова совсем.

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

ну создай новый тикет и под ним замерж. А потом за проблему бересь.

18:38пожаловаться #2

SZ

Sergey Z in rannts

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

18:39пожаловаться #3

AS

Sergey Z

а скажите мне, имеет ли хоть какой-либо смысл мержить рефакторинг, если изначально он затевался с целью решить конкретную проблему, но по окончании выясняется, что изначальная проблема не решена от слова совсем.

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

а те этот рефакторинг жизнь-разработку упростит? если да- то чего не замержить бы.

18:39пожаловаться #4

💭П

а те этот рефакторинг жизнь-разработку упростит? если да- то чего не замержить бы.

Может ещё со статусом WIP в мастер мёржить?

18:41пожаловаться #5

SZ

Sergey Z in rannts

может и так, но я очевидно не тот разработчик которому будет упрощение жизни.

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

18:41пожаловаться #6

AS

💭 Руслан Прохоров

Может ещё со статусом WIP в мастер мёржить?

не утрируй.

18:41пожаловаться #7

💭П

ИМХО:
1) Есть тикет, в нем написана задача
2) Есть PR

Если (он решает тикет) тогда мёрж иначе переделать

18:44пожаловаться #8

in

ildar nizamov in rannts

а те этот рефакторинг жизнь-разработку упростит? если да- то чего не замержить бы.

+

18:45пожаловаться #9

💭П

не утрируй.

Сори, просто на работе есть джуны со странными понятиями и отношениями к работе и под конец дня это сильно бесит и я иногда срываюсь :-(

18:45пожаловаться #10

AS

💭 Руслан Прохоров

ИМХО:
1) Есть тикет, в нем написана задача
2) Есть PR

Если (он решает тикет) тогда мёрж иначе переделать

ну тут все я бы все таки от критичности решаемых задач кода смотрел

18:45пожаловаться #11

💭П

ну тут все я бы все таки от критичности решаемых задач кода смотрел

Ну поэтому и нужно смотреть тикет.

https://youtu.be/AbSehcT19u0

18:46пожаловаться #12

RB

Roman Bolkhovitin in rannts

Sergey Z

а скажите мне, имеет ли хоть какой-либо смысл мержить рефакторинг, если изначально он затевался с целью решить конкретную проблему, но по окончании выясняется, что изначальная проблема не решена от слова совсем.

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

YouTube

Hal fixing a light bulb (from Malcolm in the Middle S03E06 - Health Scare)

I think this is pretty familiar for most of us

18:46пожаловаться #13

AS

то есть если вот это все должно быть пипец стабильно и даже если тесты прошли они могут сломать- то да, может и не надо, а вот елси это не критичный кусок и он сделает жинь легче- почему нет?!

18:46пожаловаться #14

RB

Roman Bolkhovitin in rannts

Я за вариант Антона. Рефакторинг ревьюить и мержить в рамках нового тикета

18:47пожаловаться #15

💭П

то есть если вот это все должно быть пипец стабильно и даже если тесты прошли они могут сломать- то да, может и не надо, а вот елси это не критичный кусок и он сделает жинь легче- почему нет?!

Если он делает легче одному - то не стоит, если команде то возможно. Но выше сказал своё ИМХО.

18:47пожаловаться #16

💭П

Roman Bolkhovitin

Я за вариант Антона. Рефакторинг ревьюить и мержить в рамках нового тикета

Был один тикет, сделали по нему работу, так появилился второй тикет, но первый так и не закрыт :-)

18:48пожаловаться #17

AS

💭 Руслан Прохоров

Был один тикет, сделали по нему работу, так появилился второй тикет, но первый так и не закрыт :-)

Это "плохо" для Agile? )

19:02пожаловаться #18

💭П

Это "плохо" для Agile? )

Честно не знаю, но это похоже на ИБД (Иммитация бурной деятельсности)

19:03пожаловаться #19

AS

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