VC
Я обычно все ошибки в baseline загоняю, и уже потом помаленьку чищу
да там они будут постоянно
Size: a a a
2021 January 13
КГ
так то надо в прекоммите проверку ставить и потом ещё в гитлабе делать интеграцию с бухгалтерией
Никогда не понимал когда в прекомит суют всё подряд, особенно то что выполняется по 10+ секунд
ИЛ
Никогда не понимал когда в прекомит суют всё подряд, особенно то что выполняется по 10+ секунд
ну проверять можно только коммит
ИЛ
а даже если 10 секунд, то и что?
КГ
да там они будут постоянно
Это лучше чем вообще без анализатора)
Потому что новый код придётся писать уже без ошибок, а старый как правило помаленьку вычищается
Потому что новый код придётся писать уже без ошибок, а старый как правило помаленьку вычищается
КГ
а даже если 10 секунд, то и что?
А то что если я делаю коммит, значит я хочу сделать коммит, а не запустить кучу срани.
Если я хочу запустить тесты я сделаю
Если я хочу запустить тесты я сделаю
make testИЛ
А то что если я делаю коммит, значит я хочу сделать коммит, а не запустить кучу срани.
Если я хочу запустить тесты я сделаю
Если я хочу запустить тесты я сделаю
make testа кто потом за такими честными проверять будет?
КГ
а кто потом за такими честными проверять будет?
CI?
V
а кто потом за такими честными проверять будет?
ci?
ИЛ
ci потом должен из зарплаты вычитать
VC
Это лучше чем вообще без анализатора)
Потому что новый код придётся писать уже без ошибок, а старый как правило помаленьку вычищается
Потому что новый код придётся писать уже без ошибок, а старый как правило помаленьку вычищается
там архитектура такая, что придётся всё подавлять или штуки типа
⎄делать
/** @var User */ $user = User::find(1);
⎄делать
VC
на каждой строке
VC
ci потом должен из зарплаты вычитать
только на фэйлед билдах ))
КГ
там архитектура такая, что придётся всё подавлять или штуки типа
⎄делать
/** @var User */ $user = User::find(1);
⎄делать
Не ну так маньячить конечно не нужно. Для этого baseline есть. Подавлять лучше только то где анализатор не способен нюансы языка разобрать, а не там где код не очень
VC
Не ну так маньячить конечно не нужно. Для этого baseline есть. Подавлять лучше только то где анализатор не способен нюансы языка разобрать, а не там где код не очень
там улучшения пхп типа, свои implements например, дниамическиое добавление методов и прочее
КГ
маньяки
VC
User::extend(function($model) {
$model->bindEvent('user.register', function() use ($model) {
// Code to register $model->email to mailing list
});
});
из безобидного )
VC
public $implement = [
'Backend.Behaviors.FormController',
];
а вот как анализатору обїяснить? )
КГ
public $implement = [
'Backend.Behaviors.FormController',
];
а вот как анализатору обїяснить? )
В данной строчке не понятно зачем что-то объяснять