Skip to content

Unions: Support use of TryGetValue methods with non-identity scenarios#83015

Merged
AlekseyTs merged 3 commits into
dotnet:features/Unionsfrom
AlekseyTs:Unions_37
Apr 8, 2026
Merged

Unions: Support use of TryGetValue methods with non-identity scenarios#83015
AlekseyTs merged 3 commits into
dotnet:features/Unionsfrom
AlekseyTs:Unions_37

Conversation

@AlekseyTs
Copy link
Copy Markdown
Contributor

@AlekseyTs AlekseyTs commented Apr 1, 2026

Microsoft Reviewers: Open in CodeFlow

@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @jjonescz, @dotnet/roslyn-compiler Please review

{
Debug.Assert(inputUnionType.IsUnionType);

// https://github.com/dotnet/roslyn/issues/82636: Allow conversions per spec
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this comment resolved now?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this comment resolved now?

Yes, thanks, will remove

@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler For a second review

@AlekseyTs AlekseyTs requested a review from a team April 2, 2026 12:12
@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler For a second review

Comment thread src/Compilers/CSharp/Portable/Binder/Binder_Patterns.cs Outdated
Co-authored-by: Rikki Gibson <rikkigibson@gmail.com>
@AlekseyTs AlekseyTs enabled auto-merge (squash) April 6, 2026 02:09
@AlekseyTs
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants