Ребят, нужна помощь. Нас сейчас держат от релизов три пакета:

Question

Ребят, нужна помощь. Нас сейчас держат от релизов три пакета:

definitions, di, factory. Очень нужен дополнительный ревью (я тоже планирую засесть, но у меня взгляд несколько замылен):

1. Всё ли норм с архитектурой, всё ли в тех пакетах, что надо.
2. Можно ли упростить?
3. Если ли что-то лишнее?
4. Если ли что-то неудобное?
5. Какие-то другие архитектурные косяки?

#russian #software #yii

0

03.11.2021

1 ответов

27 просмотров

Maxim Chistyakov · Accepted Answer

Да, есть фидбек. Перейдите уже на php 8 и сделайте описание типов через union types вместо Psalm. Psalm это, конечно, хорошо, но не будет подсвечиваться во всех редакторах. Непонятно зачем нужна минимальная версия 7.4 в ноябре 2021 года (когда через 18 дней выходит php 8.1). Судя по графику предыдущих версий (7.4, 7.3 и т.д.) новая версия php за 12 месяцев после релиза набирает 40% и становится самой частоиспользуемой. Т.е. в декабре 2021 версия php 8.0 будет иметь уже 40% по статистике packagist.com: https://blog.packagist.com/php-versions-stats-2021-1-edition/ Не совершайте такую же ошибку как с ::className() в Yii2. Сделайте union types средствами языка. https://github.com/yiisoft/definitions/blob/master/src/CallableDefinition.php#L67

52 похожих чатов

Ребят, нужна помощь. Нас сейчас держат от релизов три пакета:

1 ответов

Похожие вопросы