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

Wrong type hint for a module value containing a lambda #14378

Closed
kerams opened this issue Nov 23, 2022 · 2 comments · Fixed by #14552
Closed

Wrong type hint for a module value containing a lambda #14378

kerams opened this issue Nov 23, 2022 · 2 comments · Fixed by #14552
Labels
Area-LangService-Hints F# hints inspired by C# inline hints Bug good first issue Impact-Low (Internal MS Team use only) Describes an issue with limited impact on existing code.
Milestone

Comments

@kerams
Copy link
Contributor

kerams commented Nov 23, 2022

Repro steps

let func = fun () -> 3

Expected behavior

Type hint containing : (unit -> int) (with or without the parentheses).

Actual behavior

TiwQboH0iO

Related information

  • VS integration built from the current main branch

The issue originates in this function

devenv_PFgtHiFxsT

retTy is the int type, while the solution of tau is unit -> int, which is what we're looking for in the hint. I don't want to touch this, because I have no idea what ramifications it could have across the compiler.

@github-actions github-actions bot added this to the Backlog milestone Nov 23, 2022
@vzarytovskii vzarytovskii moved this to Not Planned in F# Compiler and Tooling Nov 23, 2022
@0101 0101 added Impact-Low (Internal MS Team use only) Describes an issue with limited impact on existing code. Area-LangService-Hints F# hints inspired by C# inline hints and removed Area-LangService-CodeLens labels Dec 5, 2022
@0101 0101 removed the Needs-Triage label Dec 5, 2022
@psfinaki psfinaki removed their assignment Dec 8, 2022
@psfinaki
Copy link
Member

psfinaki commented Dec 8, 2022

Unassigning myself for now in case someone else wants to fix this - can be a good exercise for contributors.

@psfinaki
Copy link
Member

psfinaki commented Jan 6, 2023

Doesn't repro anymore, adding some tests though.

@github-project-automation github-project-automation bot moved this from Not Planned to Done in F# Compiler and Tooling Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-LangService-Hints F# hints inspired by C# inline hints Bug good first issue Impact-Low (Internal MS Team use only) Describes an issue with limited impact on existing code.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants