КЧ
Ты тот код видел? Тоже бы читать не захотел
Size: a a a
2021 November 12
c
Согласен
c
Да им по должности такая роскошь недоступна
c
Я работал в одном месте, где на уровне гендира было принято решение о том что все изменения должны быть в расширениях.
АС
У нас складывается парадоксальная ситуация. Запрет на авторские комментарии прописан во внутренних стандартах. Все техлиды команд про это знают и придерживаются. Но мидлы и джуны продолжают их штамповать. Все никак не можем вычислить откуда же эта зараза распространяется
КЧ
Правильно, программисты как известно дегенераты, и не могут правильно выбрать инструмент для реализации задачи
КЧ
В Гите посмотри
c
Ну да, а если им купили правильный инструмент, то еще нужно и научить как им пользоваться.
АС
да автор сам себя в комментарии указал, чо там смотреть то. А вот кто научил - не признаются. "Ну я думал так надо делать".
КЧ
Ну так уже повод ссаными тряпками отпиздить за то что внутренний стандарт не читал
КЧ
Естественно
КЧ
А как кстати это вообще ревью проходит?
PZ
С прошлых мест работы или исторического кода.
Обычно говорят - ну у вас в коде так есть, я решил что так и надо.
А с чтением регламентов - да есть беда
Обычно говорят - ну у вас в коде так есть, я решил что так и надо.
А с чтением регламентов - да есть беда
АС
это надо, чтоб весь код читался и его нельзя было влить в ствол без ревью
КЧ
Ну а как иначе?
PZ
Да пусть вливают, но все равно вычитывать все, и просить привести к стандартам все, пусть и постфактум
PZ
Спасибо гиту, это делается очень удобно.
PZ
Как показала практика 2-3 итенрации и люди привыкают.
Но да, никто не говорил что будет просто
Но да, никто не говорил что будет просто
A
Очень понравилось помещать изменения вендорных модулей в область. Реально проще поддерживать было
КЧ
Можно же автоматизировать часть, синтакс контроль/сонар/АПК в МРах, уже часть работы заберут