Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support other PSR-20 implementations #1001

Merged
merged 3 commits into from
Feb 25, 2023
Merged

Support other PSR-20 implementations #1001

merged 3 commits into from
Feb 25, 2023

Conversation

lcobucci
Copy link
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
composer.json Outdated Show resolved Hide resolved
@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
Copy link
Collaborator

@Ocramius Ocramius left a comment

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