Skip to content

Migrate to fake project and drop .net6 dependency - #311

Merged
sergey-tihon merged 2 commits into
fsprojects:masterfrom
Wotee:migrate-to-fake-project
Oct 9, 2026
Merged

sergey-tihon merged 2 commits into
fsprojects:masterfrom
Wotee:migrate-to-fake-project

Conversation

@Wotee

@Wotee Wotee commented Oct 9, 2026

Copy link
Copy Markdown
Contributor
  • Migrates the FAKE build script to a Fake Build project.
  • Targets .NET 8 and updates FAKE packages to 6.1.4.
  • Removes the fake-cli, script lock file, and .NET 6 dependency.
  • Updates local build scripts and GitHub Actions to run the new project.

Validated by building the FAKE project and running its Build target successfully.

@Wotee

Wotee commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@CaptnCodr @sergey-tihon Here's the Fake build project update I promised in #310

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.

🔵 Needs a closer look

Multi-argument FAKE invocations can silently execute the default target instead of the requested target.

0 open findings

What changed in this PR

Migrates FAKE automation from a .NET 6 script to a .NET 8 build project.

Changes:

  • Adds a .NET 8 FAKE executable project using FAKE 6.1.4.
  • Updates local and CI build commands.
  • Removes fake-cli and the script lock file.
File Description
FsUnit.sln Removes the obsolete script reference.
build/​build.fsproj Defines the new FAKE project.
build/​build.fs Converts the script into an executable.
build.sh Runs the FAKE project on Unix.
build.cmd Runs the FAKE project on Windows.
build.fsx.lock Removes obsolete script dependencies.
.gitignore Includes the root build source directory.
.github/​workflows/​main.yml Uses only .NET 8 and the new project.
.config/​dotnet-tools.json Removes fake-cli.

🧠 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 95e668f into fsprojects:master Oct 9, 2026
3 checks passed
@sergey-tihon

Copy link
Copy Markdown
Member

Thank you @Wotee !

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.

3 participants