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

Fix usages of undefined function unwrap #111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gomzyakov
Copy link

@gomzyakov gomzyakov commented Aug 28, 2023

Function unwrap will removed in guzzlehttp/promises:2.0. Need to use Utils::unwrap instead.

@gomzyakov
Copy link
Author

@jorgeborges Hello!

Can you review this MR?

@gomzyakov
Copy link
Author

@ahmad-gelato Thanks!

@kyranb
Copy link

kyranb commented Sep 11, 2023

Is there anything blocking this from being merged?

@gomzyakov
Copy link
Author

@kyranb

No, but the repository looks abandoned

@remitmaster
Copy link

Can we get this merged?

@gomzyakov
Copy link
Author

@zydneyambat @miteshchavda @westmeadwestmead @hi-i-am-ana

Hi guys!

On the organization’s main page, everyone can see that you work at THE ICONIC. Many people need these changes, but they remain unnoticed for quite a long time. You probably know someone who can ьукпу this PR.

Can you help with this?

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

4 participants