-
-
Notifications
You must be signed in to change notification settings - Fork 127
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
Mark legacy extensions as deprecated and minor documentation improvements #234
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Thank you for looking into this, this deprecation notice is indeed long overdue (#40, #55, #62 and many others such as #230). Only added some minor remarks below, otherwise LGTM 👍
src/ExtLibevLoop.php
Outdated
@@ -12,10 +12,13 @@ | |||
use SplObjectStorage; | |||
|
|||
/** | |||
* Deprecated since v1.2.0, use [`ExtEvLoop`](#extevloop) instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should probably use a @deprecated
annotation similar to https://github.com/reactphp/event-loop/pull/231/files#diff-f1158fbec0bb446d87dbd5346fba683c58ccec1a98b1787511ec7786585e8c31R8 instead 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Thanks for the update, changes LGTM!
The maintenance for
ExtLibeventLoop
andExtLibevLoop
hasn't seen an update since 2013. It's time to move on.