phpcollection/phpcollection seems to be unmaintained about gitelephant HOT 5 CLOSED

Well, switching to array would be a much bigger task. I get started with copying the 5 files and updating the dependency, then you can have a look and make your decision.

from gitelephant.

The one feature that is actually in use from this package seems to be \PhpCollection\Sequence.

I can also imagine trying to get rid of it completely.

I might tackle this earliest starting next year, but feel free to submit a PR, I will gladly merge it if appropriate.

from gitelephant.

This looks pretty straight forward. There are 3 classes and 2 interfaces that would have to be copied into this project here:

• AbstractCollection
• AbstractSequence
• CollectionInterface
• Sequence
• SequenceInterface

And a new dependency for phpoption/phpoption would be introduced.

Is that the way to go? If so, I'm happy to put that into a PR.

from gitelephant.

...or replace it with arrays. That would be a breaking change, but as I see it, reducing such overhead that is not required for functionality is a positive thing.

Feel free to open a PR switching the composer dependency, I guess that would be fine already for now.

from gitelephant.

Created first PR and left the original comments in the files from the other package for now. Also, code style is complaining as the code was taken from the other repository. Can be addressed iof the general approach is being decided.

Turned out, that there was also a use-statement for the PhpCollection\Map class, but it seems that wasn't ever really used, so I've removed that too.

from gitelephant.