rust-lang / discord-mods-bot

discord moderation bot
MIT License
72 stars 30 forks source link

Migrate to async serenity 🎉 #98

Closed technetos closed 2 years ago

technetos commented 2 years ago

Migrating to the latest version of Serenity, adopting async, using Sqlx, oh my!

This PR looks like a alot. The majority of the changes are dropping of pub(crate) in favor of pub and adding async & .await in many places. The other signifigant change in this PR is the use of Sqlx instead of Diesel. Diesel is still a dependency in this PR because we still rely on the migrations to run automatically using Diesel. Sqlx supports automatically running migrations just like Diesel does but the format is slightly different. In this PR I chose to minimize the impact of removing Diesel by leaving the migrations alone. A follow up PR will be put together to update the migrations and use Sqlx to run them automatically.

38