-
Notifications
You must be signed in to change notification settings - Fork 9
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
fix this issue https://github.com/ayeo-flex-org/pulsar-flex/issues/90… #91
fix this issue https://github.com/ayeo-flex-org/pulsar-flex/issues/90… #91
Conversation
ping @galrose -- is this a problem with the test suite? The tests ran fine on my local machine. It appears docker-compose isn't found in CI? I see the following error at the start of the CI tests.
|
Hi @geoffhendrey If you don't mind I will fix it sometime this weekend/next week and then I will ask you to pull this changes so the CI will pass |
@danielsinai thank you very much! |
No problem @geoffhendrey |
@danielsinai merged main into my branch, all tests pass locally. You may need to approve the workflow to run the tests for this PR. |
@danielsinai I do not have permissions to merge. Can you merge this PR? |
Done @geoffhendrey thx for your contribution! |
…. Closes underlying socket and dangling timeout