-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Update dependencies from https://github.com/dotnet/arcade build 20210908.7 #58873
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer Issue DetailsNeeded for installer validation. Manually ran:
|
|
Hmm there are issues in the build. Probably a break in the Arcade SDK |
|
Ah, I didn't get the fix for that, which merged later that day. Let me pick a different build. |
|
Hmm, it looks like it had the fix dotnet/arcade@89db4b3 but needed a follow up one dotnet/arcade#7870 |
99f491e to
9eee8a5
Compare
|
The compiler errors are appearing because of a roslyn update: dotnet/arcade@ae0ffc2. cc @stephentoub |
|
@cston, @jaredpar, here's a repro: using System;
var source = new C<int>();
source.Aggregate(() => 0, (i, j) => i, (i, j) => i, i => i);
class C<T> { }
static class Extensions
{
public static TResult Aggregate<TSource, TAccumulate, TResult>(
this C<TSource> source,
Func<TAccumulate> seedFactory,
Func<TAccumulate, TSource, TAccumulate> updateAccumulatorFunc,
Func<TAccumulate, TAccumulate, TAccumulate> combineAccumulatorsFunc,
Func<TAccumulate, TResult> resultSelector) => default!;
public static TResult Aggregate<TSource, TAccumulate, TResult>(
this C<TSource> source,
TAccumulate seed,
Func<TAccumulate, TSource, TAccumulate> updateAccumulatorFunc,
Func<TAccumulate, TAccumulate, TAccumulate> combineAccumulatorsFunc,
Func<TAccumulate, TResult> resultSelector) => default!;
}That compiles fine with LangVersion 9.0. It fails with the following now on LangVersion 10.0: |
|
@stephentoub, thanks for the repro. Yes, the ambiguities are a result of the compiler changes for lambda expressions. In your repro, method type inference is inferring I will create a PR to update the tests. |
|
Disregard this PR (and deleted comment). I confused this with #58865. That seems to be progressing. Closing this. |
Needed for installer validation. Manually ran: