Skip to content
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

Merged
merged 8 commits into from Oct 29, 2023
Merged

Dotnet 8 builds #1175

merged 8 commits into from Oct 29, 2023

Conversation

TheAngryByrd
Copy link
Member

@TheAngryByrd TheAngryByrd commented Oct 12, 2023

WHAT

🤖 Generated by Copilot at 511ea16

This pull request updates the project to support the experimental net8.0 target framework, using a new environment variable BuildNet8 to control the compilation. It also fixes some JSON and line ending issues, removes unused tools, and updates dependencies.

🤖 Generated by Copilot at 511ea16

net8.0 support
added with conditions and tools
autumn of .NET

🎯🛠️🧪

WHY

Might as well see what CI says.

HOW

🤖 Generated by Copilot at 511ea16

  • Remove fake-cli and octonav tools and update paket tool to version 8.0.0-alpha002 in .config/dotnet-tools.json (link)
  • Fix JSON syntax error in .config/dotnet-tools.json (link)
  • Change label values for dotnet-version matrix in .github/workflows/build.yml and .github/workflows/test.yml to reflect current .NET versions (link, link)
  • Add BuildNet8 environment variable to .github/workflows/build.yml and .github/workflows/test.yml to enable conditional compilation of net8.0 target framework (link, link)
  • Add net8.0 to framework values in paket.dependencies to allow paket to resolve dependencies for net8.0 target framework (link, link)
  • Add System.Security.Cryptography.Pkcs package to paket.dependencies to fix runtime error when using net8.0 target framework (link)
  • Add NuGet.Versioning, NuGet.Common, and NuGet.Protocol packages to Build group in paket.dependencies to enable querying NuGet feed in build.fsx script (link)
  • Add net8.0 to TargetFrameworks values in src/FsAutoComplete.Core/FsAutoComplete.Core.fsproj, src/FsAutoComplete.Logging/FsAutoComplete.Logging.fsproj, src/FsAutoComplete/FsAutoComplete.fsproj, and test/FsAutoComplete.Tests.Lsp/FsAutoComplete.Tests.Lsp.fsproj, with a condition that checks BuildNet8 environment variable (link, link, link, link)

@TheAngryByrd TheAngryByrd force-pushed the dotnet-8-previews branch 4 times, most recently from 19fb4db to 9482101 Compare October 12, 2023 03:08
@TheAngryByrd
Copy link
Member Author

Hmm looks like 8rc2 is broke with tools dotnet/sdk#35989

Guess we'll try rc3.

@baronfel
Copy link
Contributor

@TheAngryByrd I think I've fixed the fantomas problems - they were actually because while fantomas did have a RollForward of major, the 8.x version is technically a prerelease - once I added the env var to allow prereleases to participate in rollfoward the tool began working. I think this is a UX bug and I'm going to ask Rich Lander's team to consider expanding the set of allowed SDKs/runtimes.

Copy link
Contributor

@baronfel baronfel left a 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!

@baronfel baronfel merged commit 84a3e47 into ionide:main Oct 29, 2023
12 checks passed
nojaf pushed a commit to nojaf/FsAutoComplete that referenced this pull request Nov 3, 2023
Co-authored-by: Chet Husk <baronfel@users.noreply.github.com>
Co-authored-by: Chet Husk <chusk3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants