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
Introduce TimeOut waiter #68
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.
Can you also add a mention of it in the waiters' documentation, in the docs
folder?
private $timeOut; | ||
|
||
/** | ||
* var float |
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.
Can you add a @
before var
?
*/ | ||
private $secondsEllapsed; | ||
|
||
public function __construct(Waiter $delegateWaiter, $timeOut) |
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.
Can you add a PhpDoc on the constructor?
{ | ||
$this->delegateWaiter = $delegateWaiter; | ||
$this->timeOut = $timeOut; | ||
$this->secondsEllapsed = 0; |
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.
0.0
if your secondsEllapsed
property is actually float.
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.
But we definitively should consider to switch the $seconds
in Waiter::wait
to float
also. I currently use 0.5
in my own app. And this is why I managed secondsEllapsed
in float
a9aa47a
to
5710cfa
Compare
changes done
|
Thank you @tyx! |
Let me know if any change should be done to be merged