Skip to content

Use .NET 10 SDK and migrate to slnx - #312

Merged
sergey-tihon merged 1 commit into
masterfrom
feature/dotnet-tools
Oct 9, 2026
Merged

sergey-tihon merged 1 commit into
masterfrom
feature/dotnet-tools

Conversation

@sergey-tihon

@sergey-tihon sergey-tihon commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Pin the repo and CI to the .NET 10 SDK via global.json; install .NET 8 alongside it for running net8.0 build/test apps.
  • Update local .NET tools and GitHub Actions.
  • Migrate FsUnit.sln to FsUnit.slnx and update the Fake build target.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 17:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The migration is consistent, references are updated, and CI succeeds on Windows, macOS, and Ubuntu.

0 open findings

What changed in this PR

Pins builds to .NET 10 while retaining .NET 8 runtime support and migrates the solution to .slnx.

Changes:

  • Updates SDK, local tools, and GitHub Actions.
  • Replaces FsUnit.sln with FsUnit.slnx.
  • Updates the FAKE build target.
File Description
global.json Pins .NET SDK 10.0.401.
FsUnit.slnx Defines the migrated solution.
FsUnit.sln Removes the legacy solution.
build/​build.fs Builds the .slnx solution.
.github/​workflows/​main.yml Installs required SDKs and updates actions.
.config/​dotnet-tools.json Updates repository-local tools.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@sergey-tihon
sergey-tihon merged commit db97b47 into master Oct 9, 2026
4 checks passed
@sergey-tihon
sergey-tihon deleted the feature/dotnet-tools branch October 9, 2026 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants