-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
update to net6 #133
update to net6 #133
Conversation
@baronfel Should we be updating the whole FsAutoComplete/Ionide stack to net6? Separately I noticed the solution for this project can't be opened in Visual Studio. This is because the build tasks in https://github.com/ionide/KeepAChangelog depend on FSharp.Core and that doesn't get found by .NET Framework MSBuild, e.g. error log below It's probably better just not to write the build tasks in F# tbh - or at least I think they need to be bundled with the exact version of FSharp.Core that they need. Strangely the load below is trying to load version
|
The big thing we need FAKE for at this point is doing text-replacement on the FsLibLog dependency. There are two paths to removing this need:
In other repos we've moved to using the FAKE libraries in a build.fsproj, so we'd likely do that here for ease of use/consistency. Other than that, we're good to go to net.60 - net5.0 is EOL in May so we're only slightly ahead of the curve here. |
This reverts commit 5af82ac.
This reverts commit 5af82ac.
dotnet paket update
because of error "invalid parameter 'net60' after >= or < in '== net50'"framework: net6.0
inBuild
section of paket.dependencies because of error "Error FS0039: The namespace or module 'Fake' is not defined" see Error FS0039: The namespace or module 'Fake' is not defined. fsprojects/FAKE#2619@Krzysztof-Cieslak @baronfel What are your thoughts on continued use of FAKE for repos like this?