Return to voku/portable-utf8#39
Conversation
|
Thank you @HLeithner for your very speedy review. I have now removed the deprecated |
I think not all? |
I can't find any more with a few different methods of code search. Are you able to link to where you are seeing an issue please? One of your comments may look like I missed it, as it was referring to the line after the call itself so may not have auto-resolved. |
HLeithner
left a comment
There was a problem hiding this comment.
ok, it seem github ui tricked me...
|
@HLeithner Thank you for approving. Do you have merge and release privileges or do we need to alert another maintainer? |
|
Yes I can but I would like to have some other things fixed too and move forward in php support with type hints. I created a 5.0 branch and will merge this pr into 5.0-dev, so you can reference it even if it's not tagged yet |
Reverts #33 to move from
joomla\stringback tovoku/portable-utf8. The latter is not abandoned as suggested in the previous PR, although there did seem to be some indication in the discussion there thatjoomla/stringmay be soon.The main reason I would like to do this is to address #38.
I also removed some deprecations by deleting the
CsvFileVerboseIteratorthat was only used for one language and adding#[ReturnTypeWillChange]attributes to maintain PHP 7 support.