Skip to content

Conversation

@MichalStrehovsky
Copy link
Member

Sets the default for switch added in dotnet/sdk#46375.

(Should merge before we start consuming the SDK/runtime that has this.)

Sets the default for switch added in dotnet/sdk#46375.

(Should merge before we start consuming the SDK/runtime that has this.)
@ivanpovazan
Copy link
Member

The size implications for Mono/NativeAOT MAUI iOS are provided here: dotnet/runtime#111743 (comment)

@mandel-macaque
Copy link
Contributor

/azp run xamarin-macios-pr

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mandel-macaque
Copy link
Contributor

@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.

@mandel-macaque
Copy link
Contributor

Looks like I got it to run ;)

@MichalStrehovsky
Copy link
Member Author

What's the next step for this? I don't have merge permissions here.

@rolfbjarne
Copy link
Member

@MichalStrehovsky will this have any effect if it's released in a .NET 9 service release?

@MichalStrehovsky
Copy link
Member Author

@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.

@rolfbjarne
Copy link
Member

@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)?

@rolfbjarne rolfbjarne closed this Feb 12, 2025
rolfbjarne pushed a commit that referenced this pull request Feb 13, 2025
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]>
@MichalStrehovsky
Copy link
Member Author

BTW, would you happen to know if this is needed in .NET 10 Preview 1 (or is it Preview 2+ only)?

Preview 2+ only.

@MichalStrehovsky MichalStrehovsky deleted the patch-1 branch February 13, 2025 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants