Skip to content

ApiDiff: Move TryGetRecordConstructor from GenAPI.INamedTypeSymbolExtensions up to ApiSymbolExtensions.SymbolExtensions#45801

Merged
carlossanlop merged 3 commits intodotnet:mainfrom
carlossanlop:ApiDiffPart1
Jan 9, 2025
Merged

ApiDiff: Move TryGetRecordConstructor from GenAPI.INamedTypeSymbolExtensions up to ApiSymbolExtensions.SymbolExtensions#45801
carlossanlop merged 3 commits intodotnet:mainfrom
carlossanlop:ApiDiffPart1

Conversation

@carlossanlop
Copy link
Copy Markdown
Contributor

@carlossanlop carlossanlop commented Jan 8, 2025

This PR is part of the work needed to create an ApiDiff tool that reuses some of the code from Microsoft.DotNet.GenAPI. The idea is to make the larger PR smaller and make it easier to review: #45389

This change moves an extension method from the Microsoft.DotNet.GenAPI project to the higher Microsoft.DotNet.ApiSymbolExtensions.SymbolExtensions project, so it's available more generally.

PR #45806 depends on this change being merged first.

…ymbolExtesions up to Microsoft.DotNet.ApiSymbolExtensions.SymbolExtensions.
Comment thread src/Compatibility/Microsoft.DotNet.ApiSymbolExtensions/SymbolExtensions.cs Outdated
@carlossanlop carlossanlop enabled auto-merge (squash) January 9, 2025 16:11
@carlossanlop carlossanlop removed the request for review from ericstj January 9, 2025 19:31
@carlossanlop carlossanlop merged commit babe4bd into dotnet:main Jan 9, 2025
@carlossanlop carlossanlop deleted the ApiDiffPart1 branch January 9, 2025 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Infrastructure untriaged Request triage from a team member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants