-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Replace stella-maris/clock with psr/clock #651
Conversation
@p4veI sure thing! I'll release v2.3 later this week with the newest |
Will that break any user-code that depends on this library extending |
@heiglandreas imo it's not a BC, but it might depend on the configuration in user projects. I'm using the But essentially I'd like to configure the psr interface as an alias to lcobucci/clock implementation and reduce code dependency on this package. |
You can already configure lcobucci/clock as implementation of psr/clock. you could the moment PSR20 was available on packagist. Re. the BC break: When someone uses an implementation that so far implemented an interface and drops that interface in a minor version, then the code suddenly breaks after updating to the next minor. At least according to SemVer. Introducing an interface in a minor: Fine. And while I appreciate your effort to reduce dependencies, you can only reduce your direct dependencies. You can do that by using PSR instead if the stella-maris implementation. I encourage everyone to do that. But that should not be extended to your dependencies dependencies... But that's just my (biased) 0.02 € |
@heiglandreas hmm, I'm not sure if I understand.. I guess it's possible with the update @lcobucci mentioned, once stella-maris/clock interface actually implements the psr interface (which as I see now it already does)? From what I checked when I prepared this PR it was only implementing stella-maris/clock interface which was not extending the psr interface. That's why I did this PR. I agree with the semver breaks updating though, I'm not really suggesting anything regarding to the version this should be released in, but I feel it's not really necessary to implement an interface of another package that extends the psr interface, I'd just like to see this implementing the psr interface directly. |
And I'm with you that all projects should now use Therefore the libraries that are currently extending For your use-case: Make |
5bdbd16
to
2008dac
Compare
2008dac
to
ef31bc0
Compare
Merged and released on https://github.com/lcobucci/clock/releases/tag/3.0.0 🎉 |
Hi, I was wondering if this library could replace the stella-maris/clock dependency with the newly accepted phpfig psr-20 interface https://www.php-fig.org/psr/psr-20/ for clock.