Skip to content

Remove AppVeyor, fix GitHub Actions CI for .NET 8 - #50

Merged
et1975 merged 1 commit into
masterfrom
copilot/remove-appveyor-build
Jul 29, 2026
Merged

Remove AppVeyor, fix GitHub Actions CI for .NET 8#50
et1975 merged 1 commit into
masterfrom
copilot/remove-appveyor-build

Conversation

Copilot AI commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

AppVeyor build removed; GitHub Actions is the sole CI. The workflow was also broken due to .NET 6 EOL and FAKE's MSBuild.StructuredLogger not supporting MSBuild binlog format v21 (introduced in .NET 8).

Changes

  • Deleted appveyor.yml
  • CI workflow (ci.yml): checkout@v2@v4, setup-dotnet@v1@v4, dotnet 6.0.x8.0.x
  • global.json: SDK 6.0.1008.0.100
  • tests/Elmish.UrlParser.Tests.fsproj: TargetFramework net6.0net8.0
  • build.fsx: Set DisableInternalBinLog = true on all MSBuild calls (restore, build, test, pack)

FAKE 6.1.4 bundles MSBuild.StructuredLogger built before .NET 8 shipped; it only supports binlog up to format v16 but .NET 8 MSBuild emits v21. Disabling FAKE's internal binlog reader sidesteps the incompatibility without losing build output:

DotNet.restore
    (fun opts -> { opts with MSBuildParams = { opts.MSBuildParams with DisableInternalBinLog = true } })
    projectDir

@et1975
et1975 marked this pull request as ready for review July 29, 2026 20:59
@et1975
et1975 merged commit 5eda5e8 into master Jul 29, 2026
1 of 3 checks passed
@et1975
et1975 deleted the copilot/remove-appveyor-build branch July 30, 2026 12:58
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