dotnet / 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
3.89k stars 783 forks source link

Chore: split monster-sized modules in the codebase #13653

Open psfinaki opened 2 years ago

psfinaki commented 2 years ago

We have a bunch of modules that are so big that they cause code editors to hang. The champions are probably in the compiler project, for example CheckExpressions (>10,000 LOC) or CheckDeclarations (>5,000 LOC).

We should take some monster every sprint and cut it into parts.

dsyme commented 2 years ago

See comment here https://github.com/dotnet/fsharp/pull/13652#issuecomment-1212279205 about future ways to split more