A
одно дело токсично комментировать - другое дело заапрувить код а потом пойти и переписать его
Size: a a a
2020 May 16
A
это вообще звучит еще более токсично
A
потом человек увидит что его код переписан (+ на это нужно тратить время) и у него будет вутафак фейс
SY
одно дело токсично комментировать - другое дело заапрувить код а потом пойти и переписать его
Есть ещё вариант: комментировать не токсично :) вывести на обсуждение (желательно очно) и аргументированно убедить сделать иначе
A
Это самый правильный вариант
A
просто то что предлогает автор вообще для продукта очень хреново
A
когда один пишет - неполучает нормального фитбяка - а второй тратит время на переписывания и еще нужно время свои таски делать
SY
просто то что предлогает автор вообще для продукта очень хреново
Я как-то пропустил момент, что он что-то предлагал :-[
V
Я как-то пропустил момент, что он что-то предлагал :-[
Same, только набросы на вентиль
A
No big deal if the code’s not good, I can fix it myself it I need to.A
то-есть вместо того чтобы написать - тут у нас могут быть проблемы и аргументировать
A
отвечать YES ITS GOOD а потом переписывать
SY
Да, нашёл) там чуток побольше контекст
This review I kicked off the article with? I didn’t send it. Instead I gave the guy a couple of comments and politely asked to fix a couple of things. No big deal if the code’s not good, I can fix it myself it I need to. But I can’t fix the psyche of a guy broken by dozens of harsh reviews.
SY
Т.е. он таки откаментил. Если это не критично по функциональности, а код стайл какой-нибудь, то пофиг в целом. Если потенциальная утечка по памяти, то вряд ли бы он просто заапрувил
A
переписывать чей-то чужой код в то время как ты мог бы упомянуть это в PR и обсудить это с человеком из серии -
а давай может лучше так сделаем, потому-что - довод 1, довод2 - что ты по-этому поводу думаешь вместо того чтобы аппрувить а потом идти и переписыватьA
это что за коллеги такие с которыми ты неможешь обсудить то что тебе в коде не нравится ?
SY
переписывать чей-то чужой код в то время как ты мог бы упомянуть это в PR и обсудить это с человеком из серии -
а давай может лучше так сделаем, потому-что - довод 1, довод2 - что ты по-этому поводу думаешь вместо того чтобы аппрувить а потом идти и переписыватьI can fix it myself if I need to.Кмк, тут больше про вкусовщину и читаемость. Типа, если залезет в код по случаю, то перепишет читаемей. Иначе да, странно не фидбечить чуваку, который будет везде потенциально приложуху к OOM приводить.
A
I can fix it myself if I need to.Кмк, тут больше про вкусовщину и читаемость. Типа, если залезет в код по случаю, то перепишет читаемей. Иначе да, странно не фидбечить чуваку, который будет везде потенциально приложуху к OOM приводить.
я закомитил пуллреквест что-то обсудили, его замерджили, потом на следующий день/на следующей неделе я листаю код и помню, что вот тут вот я по-другому делал и мне не нравится, как там поменяно - лезу в историю и вижу, что человек, который мне сказал, что код зашибись - лез и исправлял этот самый мой код и еще запушил это без PR
A
какая у меня будет реакция ?
A
Если я джун - а тот человек сеньор - значит он херовый ментор и к джунам его подпускать никак нельзя