mirror of
https://github.com/ramsayleung/rspotify.git
synced 2026-04-26 07:55:55 +03:00
[GH-ISSUE #78] Add support for async/await #24
Labels
No labels
Stale
bug
discussion
enhancement
good first issue
good first issue
help wanted
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/rspotify#24
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ghost on GitHub (Feb 23, 2020).
Original GitHub issue: https://github.com/ramsayleung/rspotify/issues/78
I tried to add support for
async/awaitin my branch. I added an example (examples/async-artist-top-tracks) for it and went well.If you have no future or ongoing plans to add support for
async/await, please try my branch and leave me feedback.@ramsayleung commented on GitHub (Feb 23, 2020):
As for ongoing plan to support
async/await, I have talked with the author ofspotify-tuion this PR, and everything is working in process now. Because my full time job is so damn busy, I have to take some extra time to supportasync/await, so please be patient :)@ghost commented on GitHub (Feb 23, 2020):
Sorry, i did not mean to rush you.
Understood. I'm looking forward to it. I would like to be helpful for it if I can.
@ramsayleung commented on GitHub (Feb 23, 2020):
I think your work/branch is inspired, there are a lot of things I could learn from your work. I just push my WIP work to remote origin. Instead of creating a
nonblockingpackage, I prefer to create ablockingpackage, and makenonblockingdefault.@ramsayleung commented on GitHub (Feb 23, 2020):
If you have time, you could help me to convert those endpoint functions to
async/awaitversion, I just update the internal functions toasync, there are still a lot of endpoint functions that needs to update.github.com/ramsayleung/rspotify@2ed4705679/src/spotify/client.rs (L172)github.com/ramsayleung/rspotify@2ed4705679/src/spotify/client.rs (L207)@ghost commented on GitHub (Feb 24, 2020):
will do.
@ramsayleung commented on GitHub (Feb 24, 2020):
I just submit all the work
Rspotifyneeds to do to supportasync/await, if you have time to join me, feel free to check it The workload of adding support forasync/await@ghost commented on GitHub (Feb 25, 2020):
I started to modify endpoint functions to support
async/awaitinclient.rs. I will send PR when i finished to implement them. Please wait.@ramsayleung commented on GitHub (Feb 25, 2020):
Perhaps you could send a new PR once you have finished to implement a endpoint function, or you could mark which endpoint function you are going to modify? just in case of any conflicting situation that we are both modifying the same endpoint function.
@ghost commented on GitHub (Feb 25, 2020):
oh, it has already been implemented.
@ramsayleung commented on GitHub (Feb 28, 2020):
Rspotify has come to
async/await, it's done.