Skip to content

Type checker: recover function type unification in method checking - #19036

Closed
auduchinok wants to merge 4 commits into
dotnet:mainfrom
auduchinok:tc-recover-method-functionType
Closed

Type checker: recover function type unification in method checking#19036
auduchinok wants to merge 4 commits into
dotnet:mainfrom
auduchinok:tc-recover-method-functionType

Conversation

@auduchinok

@auduchinokauduchinok commented Oct 28, 2025

Copy link
Copy Markdown
Member

Adds recovery when it's not possible to unify the method type with the expected type:

iftruethen"".ToString
Screenshot 2025-10-28 at 19 20 41

The error would lead to the method symbol not being reported, so features like 'Go to Declaration' or getting the tooltip would not work.

# Conflicts:
#	tests/FSharp.Compiler.Service.Tests/TypeChecker/TypeCheckerRecoveryTests.fs
@github-actions

github-actionsBot commented Oct 28, 2025

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/11.0.0.md

@auduchinokauduchinok changed the title Type checker: recover function type unification in method checkinType checker: recover function type unification in method checkingOct 28, 2025
@github-project-automationgithub-project-automationBot moved this from New to In Progress in F# Compiler and ToolingOct 29, 2025
@auduchinok

Copy link
Copy Markdown
MemberAuthor

This is ready 🙂

@T-Gro
T-Gro enabled auto-merge (squash) November 11, 2025 12:49
@github-actionsgithub-actionsBot added the AI-Tooling-Check-Scanned-Clean Tooling check: diff analyzed, no interesting infrastructure files label Aug 14, 2026
@abonie

Copy link
Copy Markdown
Member

@auduchinok if you could resolve conflicts on this one maybe we can merge it (or see what the build errors were)

@auduchinok

Copy link
Copy Markdown
MemberAuthor

@abonie Thanks, it turns out I have somehow done it one more time in #19314.

auto-merge was automatically disabled August 19, 2026 12:15

Pull request was closed

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in F# Compiler and ToolingAug 19, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Tooling-Check-Scanned-CleanTooling check: diff analyzed, no interesting infrastructure files

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants

@auduchinok@abonie@edgarfgp@T-Gro