feat: add phpstan for static analysis #1223
Conversation
|
Параметр |
|
Почитал статью про игнорирование прошлых ошибок, с недопуском их добавления в новом коде. С другой стороны, сразу же в начале советуют другой метод, который мне тоже приглянулся: в этой ветке исправляем ошибки уровня 0, мержим, и затем создаём ветку с исправлениями уже следующего уровня. Таким образом мы сможем постепенно дойти до желаемого уровня. |
|
Эти ошибки связаны скорее всего с тем, что phpstan немного не в курсе о Chandler
|
|
Минимизировал ошибки, их осталось лишь 17. Большинство исправил, офигел с количества неиспользованных переменных |
|
Теперь ошибок 12. Оказывается, что тут ещё был прописан код, который нигде не используется никогда бы и не сработал. А так это лучше, чем 68 ошибок |
|
|
Это наш шанс иметь тестирование без, собственно, тестов.
Используется анализатор https://phpstan.org/ с дефолтным конфигом. Советую подумать над настройкой, я этого не делал.
В силу архитектуры самого OpenVK и Chandler, анализатор запускается здесь довольно неоптимальным, на мой взгляд, образом, но это уже другая история.
Учитывая, что мы до этого никаких анализаторов не использовали, у нас отображается 100 ошибок (из-за этого CI теперь красный). Надо понять, нужно их фиксить в этом ПР, или мержить и потом уже фиксить.