Code Review Checklist
async
keyword Review library methods that use the async
keyword to see if they actually need it as it can introduce extra uncessary cost. See this gist for an example.async void
methods These are a red flag and should probably return Task
. See this post for more information.ConfigureAwait(false)
Library methods that return Task
should also call ConfigureAwait(false)
. See this article for more details.ObserveOn(RxApp.DeferredScheduler)
soon after (related to previous).