-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Psalm #51
Psalm #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Почему решил убрать phpstan?
Они же немного по разному работают. Можно оставить оба
psalm.xml
Outdated
@@ -0,0 +1,24 @@ | |||
<?xml version="1.0"?> | |||
<psalm | |||
errorLevel="7" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Почему такой низкий уровень? Он тут даже не проверяет особо возвращаемые значения
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Привет. Надо же с чего-то начинать :) Планирую итерациями увеличивать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Лучше повысь сразу уровень до нужного, добавь все ошибки что сейчас есть в baseline и потом уже рафактори. Скорее всего при итеративном подходе придется по несколько раз одно и тоже переписывать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хорошая идея. Спасибо
@tashimotor Он хуже работает с дженериками, чем psalm. У меня недавно ситуация была, когда PHPStan то видел ошибку, то не видел (разные запуски, один и тот же код). Ну и psalm на низких уровнях видит больше ошибок, чем PHPStan. Возможно дело в уровне, но мне показалось, что psalm получше справляется со своей работой, если честно :)
Принцип работы разный - это да. Но идеологически они же про одно и то же. Мне кажется, что нет смысла обе держать вместе |
No description provided.