search

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.88k stars 783 forks source link

Remove some unnecessary code introduced in #16779 #17154

Closed majocha closed 4 months ago

majocha commented 4 months ago

Some cleanup following the AsyncLocal stuff.

This code is most probably unneeded, because this now happens in Async.RunImmediate.

github-actions[bot] commented 4 months ago

:warning: Release notes required, but author opted out

[!WARNING] Author opted out of release notes, check is disabled for this pull request. cc @dotnet/fsharp-team-msft

majocha commented 4 months ago

@psfinaki, just as you noticed in the review, this is duplication. If it's green, it can be safely removed.

I think release notes not needed.