DT
это вообще дичь какая-то
Size: a a a
2020 August 11
АГ
мне думалось что если я подключу псалм сейчас, то он найдет больше ошибок, чем пхпстан. поэтому я хочу пока разгрести то, что пхпстан нашел, а потом уже псалм подключать. или у них есть взаимноконфликтные требования? по идее не должно быть
VS
ну зачем мне 20к ошибок от псалма и пхпстана, если мне пока вторых хватает?)
У меня 5к было. Все матюкались, когда я прикручивал. Сейчас все молимся на него. Сколько раз спасал...
S
это вообще дичь какая-то
при этом сабж работает да? придирки статик анализаторов - пустые угрозы
DT
мне думалось что если я подключу псалм сейчас, то он найдет больше ошибок, чем пхпстан. поэтому я хочу пока разгрести то, что пхпстан нашел, а потом уже псалм подключать. или у них есть взаимноконфликтные требования? по идее не должно быть
во первых это глупость - фиксить всю кодовую базу под псалм и пхпстан. Или у вас новый код уже не пишут и свободное время девать некуда? Делаешь бейслайн, и НОВЫЙ код уже пишеш под пхпстан и псалм. Во-вторых у них бывают непонятки насчет iterable типов, что там внутри
АГ
работает, но там очень много неиспользуемого пока в жизни кода (на будущее)
АГ
просто не нарывались еще
VS
Vlad Sobenko
У меня 5к было. Все матюкались, когда я прикручивал. Сейчас все молимся на него. Сколько раз спасал...
После этого опыта все уже были не против перехода на typesript на клиенте. Тоже очень зашло.
АГ
во первых это глупость - фиксить всю кодовую базу под псалм и пхпстан. Или у вас новый код уже не пишут и свободное время девать некуда? Делаешь бейслайн, и НОВЫЙ код уже пишеш под пхпстан и псалм. Во-вторых у них бывают непонятки насчет iterable типов, что там внутри
нового кода много. и рефакторинг активный. и тесты активно пишем новые, на имеющийся код.
АГ
надо будет так и сделать... но в целом исправить все баги - тоже надо.
KN
во первых это глупость - фиксить всю кодовую базу под псалм и пхпстан. Или у вас новый код уже не пишут и свободное время девать некуда? Делаешь бейслайн, и НОВЫЙ код уже пишеш под пхпстан и псалм. Во-вторых у них бывают непонятки насчет iterable типов, что там внутри
@return iterable<array-key, something>?АГ
чото типа, да. мне больше нравится @psalm-* аннотации
DT
нового кода много. и рефакторинг активный. и тесты активно пишем новые, на имеющийся код.
в таком случае темболее. Момент, когда ты допилишь кодовую базу до 0 ошибок стана и псалма - недостижим
KN
потому что они с иде не конфликтуют, да
АГ
в таком случае темболее. Момент, когда ты допилишь кодовую базу до 0 ошибок стана и псалма - недостижим
практически, потому что еще и рефачить основательно придется все. но я верю)
VS
надо будет так и сделать... но в целом исправить все баги - тоже надо.
Очень опасное занятие. Там может работать по стечению ошибок. Пофиксишь - перестанет работать что то.
АГ
не беда в нашем случае. пофиксим.
АГ
у нас 4 стенда)
VS
не беда в нашем случае. пофиксим.
Вы не на проде?
DT
если код работает, а псалм показывает 10к ошибок, явно не показатель того, что надо срочно исправлять код