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

Implement forward compatible Clock-Interface #506

Merged
merged 3 commits into from Apr 19, 2022

Conversation

heiglandreas
Copy link
Contributor

The stella-maris/clock package provides an interface based on the currently proposed status of PSR20. Due to the inactivity of the PSR20 working group this is a way to already provide interoperability while still maintaining forward compatibility. When the current status of PSR20 will be released at one point in time the stella-maris/clock package will extend the PSR20 interface so that this package becomes immeadiately PSR20 compatible without any further work necessary. In the long run the stella-maris/clock package will then be marked deprecated so that people can then use the PSR20 provided implementation.

Should the implementation of PSR20 change between now and a possible release then this interface will still exist and a possible implementation will need more code-changes anyhow so this interface will still provide some way of interoperability.

The stella-maris/clock package provides an interface based on the
currently proposed status of PSR20. Due to the inactivity of the PSR20 working
group this is a way to already provide interoperability while still
maintaining forward compatibility. When the current status of PSR20 will
be released at one point in time the stella-maris/clock package will
extend the PSR20 interface so that this package becomes immeadiately
PSR20 compatible without any further work necessary. In the long run the
stella-maris/clock package will then be marked deprecated so that people
can then use the PSR20 provided implementation.

Should the implementation of PSR20 change between now and a possible
release then this interface will still exist and a possible
implementation will need more code-changes anyhow so this interface will
still provide some way of interoperability.
@heiglandreas
Copy link
Contributor Author

heiglandreas commented Apr 17, 2022

There are also PRs open for this on kreait/clock-php, ergebnis/clock and beste/clock to allow cross-library adoption.

There are also PRs open for this on

to allow cross-library adoption.

This should allow the tests depending on the locked versions to execute
the right code
@lcobucci lcobucci added this to the 2.2.0 milestone Apr 19, 2022
@lcobucci
Copy link
Owner

Thanks @heiglandreas, this is a good first step towards interoperability.
We can review things when PSR-20 gets released.

I'll merge and release it, hoping the same happens on the other libs =)

@lcobucci lcobucci self-assigned this Apr 19, 2022
@lcobucci lcobucci merged commit fb533e0 into lcobucci:2.2.x Apr 19, 2022
@heiglandreas heiglandreas deleted the ImplementCLockInterface branch April 20, 2022 05:43
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.

None yet

3 participants