А
ну моё предложение немного ломает BC
Size: a a a
2020 March 04
DS
Тогда осталось тесты и ридми?
AM
Д
1. Я не уверен что readme сейчас актуален с этими изменениями.
2. AbstractContainerConfigurator выглядит очень грязно когда set() не перекрывается. Но не знаю, можно ли тут нормально. @xepozz, @roxblnfk ваши идеи бы не помешали.
3. delegateLookup() я бы всё-таки снёс. Для него ни тестов ни реального клиента...
4. ContainerProxyInterface мы хотели выносить вроде, а он ещё там.
2. AbstractContainerConfigurator выглядит очень грязно когда set() не перекрывается. Но не знаю, можно ли тут нормально. @xepozz, @roxblnfk ваши идеи бы не помешали.
3. delegateLookup() я бы всё-таки снёс. Для него ни тестов ни реального клиента...
4. ContainerProxyInterface мы хотели выносить вроде, а он ещё там.
Я бы тесты добавил и описания к классам
DS
Можно в дебаггер-вьюер такое прикрутить, я пока в yii-demo добавил
https://i.imgur.com/ZJPkrIN.png
https://i.imgur.com/ZJPkrIN.png
DS
Д
не важно как, главное брались из дебагер->гетИд
Д
но да, так :)
T
We should add all exceptions at https://github.com/yiisoft/friendly-exception, instead of each package repeating the same exceptions.
Д
TradersVE
We should add all exceptions at https://github.com/yiisoft/friendly-exception, instead of each package repeating the same exceptions.
No, this repo not for all exceptions, only for *Friends*. This exceptions will provide the solution guide how to solve the problem
А
Only for interface
T
We must create a repository of exceptions, to have a standard within the framework for them.
А
Can you give examples of errors that need to be placed in a separate package?
T
No, this repo not for all exceptions, only for *Friends*. This exceptions will provide the solution guide how to solve the problem
It is that all exceptions should show the error message of the problem.
T
Can you give examples of errors that need to be placed in a separate package?
Do a review of all packages and you will find repeated exceptions used by all packages.
А
TradersVE
It is that all exceptions should show the error message of the problem.
Friendly exceptions also show solutions for developers
А
TradersVE
Do a review of all packages and you will find repeated exceptions used by all packages.
The essence of these exceptions is that they show their origin with their namespace
А
This is a grouping factor by which you can catch exceptions.