Carbon is not mandatory, we can use php DateTime objects

[stephpy]

Hello,

Carbon should not be a mandatory requirements since PHP provides DateTimeInterface since php >5.5

I let a test with your carbon implementation to prove it.

This version of carbon has itself some requirements (Symfony <7.0) which makes me unconfortable that's why I checked if it's really needed here. It's not the case.

Regards

[rubenvanassche]

Take a look at TimezoneRangeCollection it still has Carbon instances

[stephpy]

I'm sorry... It was a big mistake ...

I replaced it with Backward Compatibility.

Not sure it's a good idea to let the test with Carbon implementation because here we don't use any specific carbon feature since i replaced Carbon::minValue Carbon::maxValue by private methods which reproduce Carbon implementation.

Just say me if you want me to delete the test with Carbon implementation.

[rittou]

@freekmurze, I hope you had a great holiday!
I just wanted to know if we can speed up this PR.

[rittou]

Sorry, but anyone is here? 🆘

[Isengo1989]

Any news here?

@stephpy - seems like you resolved the two raised issues from @freekmurze but not import the namespace / switch from private to protected?

[stephpy]

@Isengo1989 I did both :)

[rubenvanassche]

Thanks!