-
Notifications
You must be signed in to change notification settings - Fork 549
Set default for UseSizeOptimizedLinq #22074
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
Sets the default for switch added in dotnet/sdk#46375. (Should merge before we start consuming the SDK/runtime that has this.)
|
The size implications for Mono/NativeAOT MAUI iOS are provided here: dotnet/runtime#111743 (comment) |
|
/azp run xamarin-macios-pr |
|
No pipelines are associated with this pull request. |
|
@MichalStrehovsky @ivanpovazan lets way for @rolfbjarne but it looks of with me. We need to create this as a branch of the repo due to the new limitations in the CI. I'll create one and will link it to this PR. |
|
Looks like I got it to run ;) |
|
What's the next step for this? I don't have merge permissions here. |
|
@MichalStrehovsky will this have any effect if it's released in a .NET 9 service release? |
Nothing will read this property in .NET 9 so there should be no effect. |
Great, thanks, I'll take over this and get it in (with PR #22076, so I'm closing this). BTW, would you happen to know if this is needed in .NET 10 Preview 1 (or is it Preview 2+ only)? |
Sets the default for switch added in dotnet/sdk#46375. (Should merge before we start consuming the SDK/runtime that has this.) Supersedes PR #22074 so that we can run the CI /cc @MichalStrehovsky @ivanpovazan --------- Co-authored-by: Michal Strehovský <[email protected]>
Preview 2+ only. |
Sets the default for switch added in dotnet/sdk#46375.
(Should merge before we start consuming the SDK/runtime that has this.)