-
Notifications
You must be signed in to change notification settings - Fork 156
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
Dotnet 8 builds #1175
Dotnet 8 builds #1175
Conversation
19fb4db
to
9482101
Compare
9482101
to
643def3
Compare
Hmm looks like 8rc2 is broke with tools dotnet/sdk#35989 Guess we'll try rc3. |
@TheAngryByrd I think I've fixed the fantomas problems - they were actually because while fantomas did have a RollForward of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing the plumbing here!
Co-authored-by: Chet Husk <[email protected]> Co-authored-by: Chet Husk <[email protected]>
WHAT
🤖 Generated by Copilot at 511ea16
This pull request updates the project to support the experimental
net8.0
target framework, using a new environment variableBuildNet8
to control the compilation. It also fixes some JSON and line ending issues, removes unused tools, and updates dependencies.🤖 Generated by Copilot at 511ea16
🎯🛠️🧪
WHY
Might as well see what CI says.
HOW
🤖 Generated by Copilot at 511ea16
fake-cli
andoctonav
tools and updatepaket
tool to version8.0.0-alpha002
in.config/dotnet-tools.json
(link).config/dotnet-tools.json
(link)label
values fordotnet-version
matrix in.github/workflows/build.yml
and.github/workflows/test.yml
to reflect current .NET versions (link, link)BuildNet8
environment variable to.github/workflows/build.yml
and.github/workflows/test.yml
to enable conditional compilation ofnet8.0
target framework (link, link)net8.0
toframework
values inpaket.dependencies
to allowpaket
to resolve dependencies fornet8.0
target framework (link, link)System.Security.Cryptography.Pkcs
package topaket.dependencies
to fix runtime error when usingnet8.0
target framework (link)NuGet.Versioning
,NuGet.Common
, andNuGet.Protocol
packages toBuild
group inpaket.dependencies
to enable querying NuGet feed inbuild.fsx
script (link)net8.0
toTargetFrameworks
values insrc/FsAutoComplete.Core/FsAutoComplete.Core.fsproj
,src/FsAutoComplete.Logging/FsAutoComplete.Logging.fsproj
,src/FsAutoComplete/FsAutoComplete.fsproj
, andtest/FsAutoComplete.Tests.Lsp/FsAutoComplete.Tests.Lsp.fsproj
, with a condition that checksBuildNet8
environment variable (link, link, link, link)