Skip to content

Support other PSR-20 implementations#1001

Merged
lcobucci merged 3 commits into
5.0.xfrom
use-clock-from-psr
Feb 25, 2023
Merged

Support other PSR-20 implementations#1001
lcobucci merged 3 commits into
5.0.xfrom
use-clock-from-psr

Conversation

@lcobucci

Copy link
Copy Markdown
Owner

This promotes better interoperability by allowing users to plug any Clock implementation that follows PSR-20 - we still fallback to lcobucci/clock.

@lcobucci lcobucci added this to the 5.0.0 milestone Feb 21, 2023
@lcobucci lcobucci self-assigned this Feb 21, 2023
Comment thread composer.json Outdated
@lcobucci
lcobucci force-pushed the use-clock-from-psr branch 2 times, most recently from 812ddd9 to eb8bca5 Compare February 23, 2023 21:22
@lcobucci
lcobucci requested a review from Slamdunk February 23, 2023 21:40

@Ocramius Ocramius left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

This promotes better interoperability by allowing users to plug any
Clock implementation that follows PSR-20 - we still fallback to
lcobucci/clock.

Signed-off-by: Luís Cobucci <[email protected]>
This allows users to choose which PSR-20 they want to have installed in
their systems, without changing our test suite that much.

Signed-off-by: Luís Cobucci <[email protected]>
@lcobucci
lcobucci merged commit 47bdb0e into 5.0.x Feb 25, 2023
@lcobucci
lcobucci deleted the use-clock-from-psr branch February 25, 2023 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants