А
там вроде хотели на csfixer его поставить
:-/
Size: a a a
2020 March 23
Д
А
руками надёжнее
Д
😁 а разница?)
А
😁 а разница?)
когда ты делаешь руками, то ты уверен в том, что этот класс чётко работает в этом режиме. Фиксер навтыкает strictы, а мы потом ходи проверяй
Д
делаешь тесты, не ставишь стрикт мод => фиксер ставит => тесты падают => pr ждет, пока пофиксят тесты
А
тесты не всегда всё покрывают и потом всплывёт у конечного пользователя, он пойдёт писать, что yii гмно, вы мне всё сломали
Д
я же сам кроме тестов ничего не делаю)
Д
в плане проверки
А
у нас срач на эту тему дольше, чем если бы ты 5 раз поставил эти стрикты
А
function convertData($data): StreamInterface может как-то convert?DataToStream?
Д
что решаем по RequestPaginatorTag?
AM
сносим
А
в AbstractResponseFactory если
streamFactory есть protected, то нужен ли function createStream(string $content):?AM
в текущем виде уж точно не нужен
Д
в AbstractResponseFactory если
streamFactory есть protected, то нужен ли function createStream(string $content):?возможно нет, оставил protected для создания других типов стрима
А
возможно нет, оставил protected для создания других типов стрима
там же в конструкторе можно задать
А
нужную фабрику
А
т.е. предлагаю либо $streamFactory приватить, либо createStream() убирать