Skip to content
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

Symbols: add Mfv.ApparentEnclosingType #17494

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

auduchinok
Copy link
Member

Adds another way of getting extended type which allows inspecting extended type parameter types.

@auduchinok auduchinok requested a review from a team as a code owner August 6, 2024 12:08
Copy link
Contributor

github-actions bot commented Aug 6, 2024

❗ Release notes required


✅ Found changes and release notes in following paths:

Change path Release notes path Description
src/Compiler docs/release-notes/.FSharp.Compiler.Service/9.0.200.md

@auduchinok auduchinok force-pushed the mfv-apparentContainingType branch 2 times, most recently from 09529b9 to ed5736b Compare August 6, 2024 12:20
@T-Gro T-Gro marked this pull request as draft August 14, 2024 17:44
@T-Gro
Copy link
Member

T-Gro commented Aug 14, 2024

Converted to draft until green, please mark as ready to review once this is ready to be merged.

@auduchinok auduchinok force-pushed the mfv-apparentContainingType branch from 962b1a4 to 6658d5f Compare September 2, 2024 13:56
@auduchinok auduchinok force-pushed the mfv-apparentContainingType branch 2 times, most recently from a55efd7 to 6eb8bca Compare September 19, 2024 12:35
@auduchinok auduchinok force-pushed the mfv-apparentContainingType branch from 6eb8bca to 654ce91 Compare September 19, 2024 12:38
@auduchinok
Copy link
Member Author

@vzarytovskii Could someone from MS team look at the changes needed in Editor project, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New
Development

Successfully merging this pull request may close these issues.

2 participants