Size: a a a

2021 February 10

IC

Ilya Chernoudov in pro.net
по моему - обычная практика
источник

E

EgorBo in pro.net
Denis Petukhov
Не обязательно. Чтото могло устреть итд. Можно жи оспорить и поменять
ага, заслать ПР на рефакторинг огромной базы с нулевым профитом и усложнить всем гит блейм, породить конфликты с другими ПР
источник

AH

Ayrat Hudaygulov in pro.net
Denis Petukhov
var станет последним гвоздем в гроб)
Там автор скорее принимал странные технические решения, типа выбор умирающего сервис фабрика, отсутствие тестов на логику, спагетти код и плохой деплой менеджмент. По итогу ебанутое количество проблем в продакшне. Все настолько плохо что у людей беклог по написанию пост мортемов - я не шучу.

Они не то что чинить не успевают, они даже репорты не успевают написать. И фичи туда же.


Я попытался улучшить ситуацию, на код ревью этот мудак к var приебался у переменной которая на следующей строчке деконстрактилась в патерн матчинге.

Сукабля, у тебя проект горит, а ты мудак на ревью хуйней страдаешь
источник

DP

Denis Petukhov in pro.net
У меня в прошлой компании был репозиторий с натсройками решарпера
источник

DP

Denis Petukhov in pro.net
Хочешь менять кодстайл делаешь PR etc
источник

SE

Stanislav Ershov in pro.net
Denis Petukhov
Хочешь менять кодстайл делаешь PR etc
источник

IC

Ilya Chernoudov in pro.net
если почему то команда использует какой то уебанский плагин для форматирования - то ты тоже используй, пример
источник

E

EgorBo in pro.net
Ayrat Hudaygulov
Там автор скорее принимал странные технические решения, типа выбор умирающего сервис фабрика, отсутствие тестов на логику, спагетти код и плохой деплой менеджмент. По итогу ебанутое количество проблем в продакшне. Все настолько плохо что у людей беклог по написанию пост мортемов - я не шучу.

Они не то что чинить не успевают, они даже репорты не успевают написать. И фичи туда же.


Я попытался улучшить ситуацию, на код ревью этот мудак к var приебался у переменной которая на следующей строчке деконстрактилась в патерн матчинге.

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

SE

Stanislav Ershov in pro.net
а нахуя простите? .editorconfig еще не поддерживался?
источник

E

EgorBo in pro.net
одно дело спорить за реализацию или фикс

другое за минорный кодстайл
источник

DP

Denis Petukhov in pro.net
Stanislav Ershov
а нахуя простите? .editorconfig еще не поддерживался?
Ну можешь так же хранить его в общей репе
источник

SE

Stanislav Ershov in pro.net
Denis Petukhov
Ну можешь так же хранить его в общей репе
ну да, студия же его применяет к проекту
источник

SE

Stanislav Ershov in pro.net
поэтому какие проблемы к настройкам?
источник

DP

Denis Petukhov in pro.net
Речь не про один проект
источник

DP

Denis Petukhov in pro.net
А кодстайл на уровне компании
источник

VS

Vasily Shapenko in pro.net
EgorBo
одно дело спорить за реализацию или фикс

другое за минорный кодстайл
Проблема в том, что годный фикс заворачивают из-за минорного код стайла
источник

DP

Denis Petukhov in pro.net
Типа дефолтный вот такой
источник

AH

Ayrat Hudaygulov in pro.net
EgorBo
ну так изменил бы как ему нравится, что за желание показать кто тут альфач
Ой фпесду, я не хочу работать с идиотами и ушёл оттуда на бутылку онкола в текущую команду. Тут кодобаза топчик
источник

E

EgorBo in pro.net
Ayrat Hudaygulov
Ой фпесду, я не хочу работать с идиотами и ушёл оттуда на бутылку онкола в текущую команду. Тут кодобаза топчик
см правило "не работай с долбаебами" да
источник

DP

Denis Petukhov in pro.net
Конечно за кодстайл уж плохо шеймить прям
источник