corvus-dotnet / corvus-dotnet/Corvus.Testing

Does the XUnit demo project need to use ConfigureAwait?

Open
#250 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
13
Forks
11
PR merge metrics
No merged PRs in 30d

Description

We're getting a load of warnings from Roslynator about missing `ConfigureAwait` calls in the Xunit.Demo project.

We need to determine whether they are in fact unneeded (e.g., as in most ASP.NET Core projects) and we should therefore surpress the message, or if they are in fact required.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the Xunit.Demo project and the Roslynator warnings for missing ConfigureAwait calls. Determine whether ConfigureAwait is required in this project, using the ASP.NET Core comparison in the issue as context. Done means reaching a decision and either suppressing the warning or addressing the required calls.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.