safesparrow / fsharp

The F# compiler, F# core library, F# language service, and F# tooling integration for Visual Studio
https://dotnet.microsoft.com/languages/fsharp
MIT License
1 stars 0 forks source link

Pairs #32

Closed nojaf closed 1 year ago

nojaf commented 1 year ago

Hi @safesparrow, I was able to rework the code to have a new version of the infamous fsix approach. I went with the names Pair and Physical and transform the resolved graph after mkGraph. This has quite the appeal to me as it is more of an implementation detail isolated from the graph resolution.

I wrote some comments on why we do this, for future readers. Please take a look at this and let me know if it makes sense.