fn that returns a future and contains a single async block should be linted to use async fn instead

[khuey]

Bounced over from rust-lang/rust#71391.

Something like

fn foo(args) -> impl Future<Output = u32> {
  async move {
    // Do stuff with args.
    my_u32
  }
}

async fn is strictly better.