-
Notifications
You must be signed in to change notification settings - Fork 39
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
Try support FSharp 7 #190
Try support FSharp 7 #190
Conversation
@rstm-sf This is in draft, but since the build actually passed, is F# part ready to merge? |
Was just looking for this. Would love to see it get merged soon :) |
.github/workflows/dotnet.yml
Outdated
dotnet-version: 6.0.x | ||
- uses: actions/setup-dotnet@v1 | ||
with: | ||
dotnet-version: 7.0.x |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Sorry for delay. FCS used to depend on msbuild (https://github.com/dotnet/msbuild/tree/main/src/Utilities#netstandard20-target), but the dependency has been removed from the 43.7.200 version https://www.nuget.org/packages/FSharp.Compiler.Service/43.7.200#dependencies-body-tab |
Having done these in the past, this one LGTM 👍 |
# Conflicts: # AspNetCore.Demo.Net6/packages.lock.json # Tests.Roslyn44/packages.lock.json # Tests.RoslynLatest/packages.lock.json
@ashmind just ping |
when will this be merged? |
Thanks for your work! Unfortunately I wasn't able to support it promptly enough due to other priorities. Therefore I'll close this one. |
No description provided.