Paulo Morgado

This should be the API: Task FlushAsync(CancellationToken); You can `await` it, `Wait` it or cancel it.

The directory where the coverage results will be published is required to exist. Otherwise it throws: Data collector 'Code Coverage' message: Data collector caught an exception of type 'Microsoft.VisualStudio.Coverage.VanguardException': 'Running...

> @JVimes You can use `true` for build tools that don't have any TargetFramework dependent references. It will modify the nuspec to remove the targets, allowing any consumer to reference...

But that still defines a TFM. Right, @baronfel?

@roji, I thought of that, but `ArraySegment` has explicit `Offset` while that might not be possible to obtain from a `Memory`. The user can try getting an `ArraySegment` from a...

No. And in those corner cases, it's valid that the user needs to provide a memory instance that wraps an array. But you'll still end up with an `ArraySegment`, because...

I would expect that `new S()` could be different from `default(S)`. That means that developers should be aware that their struct can be created without running any constructor.

Call me strange, but I still don't feel comfortable with optional arguments in public APIs. To the proposal in question, the callee shouldn't be required to know anything about the...

There could be an helper type that takes all the values in the constructor as optional arguments: ```c# public static void Assert(AssertContext context, bool condition); public static void Assert(AssertContext context,...

@alrz, are you expecting the extension method to win over the class method?