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

Default async/await implementation #37

Merged
merged 3 commits into from Feb 4, 2022

Conversation

laqiguy
Copy link
Contributor

@laqiguy laqiguy commented Jan 31, 2022

There is no need to do it yourself. Default implementation uses already implemented methods with closures and wrap them into async/await with cancelation.

@vani2 vani2 self-requested a review January 31, 2022 11:04
Sources/Apexy/AsyncAwaitHelper.swift Outdated Show resolved Hide resolved
Sources/Apexy/Client.swift Outdated Show resolved Hide resolved
@laqiguy
Copy link
Contributor Author

laqiguy commented Feb 3, 2022

@vani2 Please, look into changes your comments caused.

# Conflicts:
#	Sources/ApexyAlamofire/AlamofireClient.swift
@laqiguy laqiguy merged commit 38b4081 into RedMadRobot:master Feb 4, 2022
@laqiguy laqiguy deleted the fix/async-methods branch July 23, 2022 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants