Skip to content

Async await#44

Open
Andrewangeta wants to merge 14 commits intomainfrom
async-await
Open

Async await#44
Andrewangeta wants to merge 14 commits intomainfrom
async-await

Conversation

@Andrewangeta
Copy link
Copy Markdown
Member

@Andrewangeta Andrewangeta commented Feb 20, 2022

Fixes #42

Starting to rewrite everything is async await.
This includes rewriting how credential loading works.
It's more accurate according to spec which it wasn't before.
And it's easier to use and has a more friendly API.
APIs to be converted:

  • SecretsManager
  • Translation
  • DataStore
  • PubSub
  • CloudStorage

@nathanfallet
Copy link
Copy Markdown
Member

@Andrewangeta Is this PR ready for review?

@Andrewangeta
Copy link
Copy Markdown
Member Author

@nathanfallet Not quite. I really should take some time this week to finish up the implementation. If you want, you can use the current branch or even a specific commit hash. I've been using the branch for a while for cloud storage.

@nathanfallet
Copy link
Copy Markdown
Member

There are conflicts so you might need to merge/rebase before continuing (or merging in the main branch). Keep us updated if you go further, that would be a great improvement.

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.

Support of swift 5.5 async feature

2 participants