corvus-dotnet / corvus-dotnet/Corvus.Leasing

Monitor whether ConfigureAwait(false) still problematic in specs

Open
#61 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
3
Forks
1
PR merge metrics
No merged PRs in 30d

Description

In https://github.com/corvus-dotnet/Corvus.Extensions/pull/53#pullrequestreview-344062016 @mwadams asked me to remove all `ConfigureAwait(false)` calls in specs. However, on further discussion, we now suspect this is no longer required. So we're going to try without it, and see how it goes. This work item is to keep track of the fact that we're monitoring this.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the PR #53 review discussion linked in the issue, then inspect the specs for ConfigureAwait(false) calls. Run the relevant specs with and without those calls; done requires a clear decision about whether they remain problematic and a documented outcome.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
testing
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
15/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.