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

Add v1 #35

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add v1 #35

wants to merge 4 commits into from

Conversation

iggyvolz
Copy link

@iggyvolz iggyvolz commented Dec 2, 2021

Updated dependencies to Amp v3 similar to amphp/http-client-psr7#4 - don't have it working yet because I'm getting some fiber errors that I haven't been able to debug trying to run the examples:

Fatal error: Uncaught Error: Scheduler suspended or exited unexpectedly in /home/katie/beanstalk/vendor/revolt/event-loop/src/EventLoop/Internal/DriverSuspension.php on line 85

Seems to be related to the $deferred->getFuture()->await(); call in BeanstalkClient::send() but I can't figure out for the life of me what I'm doing wrong there.

@iggyvolz iggyvolz marked this pull request as ready for review December 15, 2021 20:07
*/
public function testReserve($reserveTimeout, $connectionCloseTimeout, $testFailTimeout) {
public function testReserve(?int $reserveTimeout, int $connectionCloseTimeout, int $testFailTimeout): void {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason this is always failing on the first invocation with no exception thrown, but passing (i.e. throwing correctly) on the second invocation

@iggyvolz
Copy link
Author

Finally got it working with the exception of one test that I can't figure out - this should fix #12 and #13 as well

@kelunik
Copy link
Member

kelunik commented Dec 15, 2021

@iggyvolz Thank you for your work! We'll eventually get to this package, we're currently tagging beta versions for all the lower level libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants