Florian Verdonck
Florian Verdonck
I agree that the default value is a bit weird here. We should document the `root` parameter workaround.
>sounds good @nojaf ? Yup!
Hmm, this does seem like a rather new area when it comes to the existing test projects. I'm not sure really, you could create a new project I suppose. On...
Well, the whole solution is rather arcane, isn't it? Anyway, I don't think all test files are that bad. You could also consider using https://github.com/TestableIO/System.IO.Abstractions in your implementation. That could...
This rings a bell. I believe this is a change with `NicePrint` over in the F# compiler. I remember some time ago that I wanted to include parameter attributes in...
I think extending the `DisplayEnvironment` (in https://github.com/dotnet/fsharp/blob/1e78d396a1397009fe52d609660376f0ce5c5d52/src/Compiler/Symbols/SymbolHelpers.fs#L546-L551) and only add the attributes when our new flag is enabled (Somewhere in https://github.com/dotnet/fsharp/blob/4934588ae8dfac3de2e8ce58198601edbfddfd08/src/Compiler/Checking/NicePrint.fs#L1047) makes the most sense. We would disable it for...
> Sure, do you intend to stream this? We could but that is really up to you. I wasn't specifically proposing it with that in mind.
Not sure if this helps but in Fantomas, we switched to local assemblies instead of `#r "nuget:`: https://github.com/fsprojects/fantomas/blob/v6.0/docs/docs/end-users/Configuration.fsx#L18-L19
Hello @teo-tsirpanis, thank you for opening this issue. As a general remark, I would be on board with this. Alas, actually doing this would be a bit more complicated. We...
In https://github.com/fsprojects/FSharp.Formatting/pull/796 we don't use `Microsoft.Build.Locator`, but the MSBuild assemblies are no longer bundled into the CLI tool.