ClickhouseClient icon indicating copy to clipboard operation
ClickhouseClient copied to clipboard

Published 20 hours ago • verified publisher icon the-tinderbox

Deprecated fix

Open trin4ik opened this issue 2 years ago • 2 comments

Привёл объявление методов к новому синтаксису, а то deprecated ругается в тинкере. Только начал юзать пакет, возможно где-то ещё есть подобные задачи, предлагаю оставить pr пока как есть, дополню со временем, если ещё где найду.

trin4ik avatar Jun 19 '23 22:06 trin4ik

Убрал все ненужные phpdoc типы, поднял php до 8 версии, проверил все тесты. Видимо, надо пускать 4-ой веткой, чтобы не сломать совместимость, т.к. mixed типы и объявление типов у свойств классов появились только в 8+.

Так же по ходу поправил пару багов, буквально косметику.

trin4ik avatar Jun 20 '23 14:06 trin4ik

Thanks, hope your PR will be accepted soon. I tried to upgrade my project to use PHP 8.1 but reached an issue with the return type being incorrect with ArrayAccess class

phuongdm1987 avatar Jun 26 '23 04:06 phuongdm1987