Skip to content

Symbols: try to use ValReprInfoForDisplay in Mfv.CurriedParameterGroups - #18124

Merged
T-Gro merged 4 commits into
dotnet:mainfrom
auduchinok:symbols/mfv-reprInfo
Dec 18, 2024
Merged

Symbols: try to use ValReprInfoForDisplay in Mfv.CurriedParameterGroups#18124
T-Gro merged 4 commits into
dotnet:mainfrom
auduchinok:symbols/mfv-reprInfo

Conversation

@auduchinok

@auduchinokauduchinok commented Dec 9, 2024•
edited
Loading

Copy link
Copy Markdown
Member

Fixes#16542.

@auduchinok
auduchinok requested a review from a team as a code ownerDecember 9, 2024 17:33
@github-actions

github-actionsBot commented Dec 9, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

❗ Release notes required

✅ Found changes and release notes in following paths:

Change pathRelease notes pathDescription
src/Compilerdocs/release-notes/.FSharp.Compiler.Service/9.0.200.md

@auduchinok
auduchinokforce-pushed the symbols/mfv-reprInfo branch 3 times, most recently from c329e9a to 6be90f6CompareDecember 10, 2024 14:19
edgarfgp
edgarfgp reviewed Dec 11, 2024
View reviewed changes
Comment threadtests/FSharp.Compiler.Service.Tests/Symbols.fs
@auduchinok

auduchinok commented Dec 18, 2024

Copy link
Copy Markdown
MemberAuthor

Can this be reviewed, please? :)

@T-Gro
T-Gro enabled auto-merge (squash) December 18, 2024 19:42
@T-Gro
T-Gro merged commit bae73d3 into dotnet:mainDec 18, 2024
@auduchinok
auduchinok deleted the symbols/mfv-reprInfo branch December 19, 2024 12:43
This was referenced Jan 8, 2025
Closed
Closed
Closed
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vzarytovskiivzarytovskiivzarytovskii approved these changes
@T-GroT-GroT-Gro approved these changes
+1 more reviewer
@edgarfgpedgarfgpedgarfgp approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

F# Compiler and Tooling
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Symbols: local functions don't provide parameter names and declaring type info

4 participants

@auduchinok@vzarytovskii@edgarfgp@T-Gro