spacedentist / spr

Submit pull requests for individual, amendable, rebaseable commits to GitHub
https://getcord.github.io/spr/
MIT License
378 stars 33 forks source link

Move to tokio runtime #87

Closed sven-of-cord closed 2 years ago

sven-of-cord commented 2 years ago

While I like some ideas behind the "smol" runtime and ecosystem, it is clearly less popular then the tokio run-time. For example, many libraries that perform work asynchronously assume the tokio runtime, which meant we had to linked against it anyway and had to call .compat() on the returned futures to make them awaitable with our smol-based runtime. Moving from smol to tokio is not even that complicated:

Moving to tokio reduces friction, let's do it!

Test Plan: I used spr diff --all on a branch with multiple local commits in a test repo.