Size: a a a

2020 December 21

DP

Dmitri Ponomarjov in PHP
но так всё равно лучше, чем никак
источник

VC

Vladimir Chernyshev in PHP
плюс ошибки типа
 Line   vendor/twig/twig/lib/Twig/TokenParser/Set.php                                    
------ ---------------------------------------------------------------------------------
 11     Class Twig_TokenParser_Set extends final class Twig\TokenParser\SetTokenParser.
источник

DP

Dmitri Ponomarjov in PHP
vendor весь лучше эксклюдить, вендор не в вашей ответственности лежит
источник

VC

Vladimir Chernyshev in PHP
А он полностью его игнорит или сигнатуры таки смотрит для проверки как мы его вызываем?
источник

DP

Dmitri Ponomarjov in PHP
Я не эксперт по псалму, но, по-моему, смотрит потому что вызов - часть проверяемого кода. Проверить это легко.
источник

DP

Dmitri Ponomarjov in PHP
вообще считаю, что на packagist надо делать встроенный psalm-check, который будет у каждого релиза каждого пакета показывать число ошибок
источник

DP

Dmitri Ponomarjov in PHP
и в композере выдавать warning если ошибок больше некоего порога
источник

DP

Dmitri Ponomarjov in PHP
это качеству кода всего коммьюнити помогло бы очень-очень
источник

ЕР

Евгений Ромашкан... in PHP
Vladimir Chernyshev
А он полностью его игнорит или сигнатуры таки смотрит для проверки как мы его вызываем?
Полностью смотрит, но игнорит ошибки(как и во всём другом заигноренном коде)
источник

ЕР

Евгений Ромашкан... in PHP
Мне казалось, правда, что там в дефолтном конфиге vendor уже в игноре
источник

DP

Dmitri Ponomarjov in PHP
Евгений Ромашкан
Мне казалось, правда, что там в дефолтном конфиге vendor уже в игноре
В легаси папка вендор бывает в самых неожиданных местах :)
источник

VC

Vladimir Chernyshev in PHP
Dmitri Ponomarjov
В легаси папка вендор бывает в самых неожиданных местах :)
их ещё может быть несколько именно vendor со своими composer.json, а ещё могут быть всякие либы и плагины просто закомиченные
источник

ЕР

Евгений Ромашкан... in PHP
Vladimir Chernyshev
их ещё может быть несколько именно vendor со своими composer.json, а ещё могут быть всякие либы и плагины просто закомиченные
Все в игнор кидай)
источник

VC

Vladimir Chernyshev in PHP
Алексей Гевондян
тогда не мало) на максималках будет раза в 3 больше
[ERROR] Found 9863 errors  с игнором вендора... Без игнора 75000+
источник

VC

Vladimir Chernyshev in PHP
968 - магические фасады ларавеля навскидку... копать - не перекопать...
источник

VS

Vlad Sobenko in PHP
Vladimir Chernyshev
[ERROR] Found 9863 errors  с игнором вендора... Без игнора 75000+
Ну норм. Вообще поставить просто псалм на сорс папки и всё.
источник

VC

Vladimir Chernyshev in PHP
это на сорс папках
источник

ЕР

Евгений Ромашкан... in PHP
errorLevel понижать можно
источник

ЕР

Евгений Ромашкан... in PHP
Плюс для ларавеля есть псалм плагин, вроде люди юзают
источник

VS

Vlad Sobenko in PHP
Евгений Ромашкан
errorLevel понижать можно
Я бы не советовал.
Бейсланим на дефолтном левеле. Месяц-второй обвыкаемся с псалмом -> врубаем макс. лвл, бейслайним - радуемся.
источник