Dotmim.Sync icon indicating copy to clipboard operation
Dotmim.Sync copied to clipboard

Use ConfigureAwait(false) everywhere

Open symbiogenesis opened this issue 3 months ago • 0 comments

In the .editorconfig file the Code Analyzer rule CA2007 should be set to the severity of "error"

Library code such as this project should use ConfigureAwait(false) everywhere. Even though it is ugly, this helps improve performance. And it is already being done mostly. Just not consistently.

The rule will help ensure this never regresses.

The only exceptions would be in the tests and in Samples which have a UI. Those could have their own .editorconfig files to override the root config.

In the tests, actually the opposite of CA2007 would be ideal, since XUnit doesn't like ConfigureAwait(false)

symbiogenesis avatar Apr 10 '24 16:04 symbiogenesis