Skip to content
This repository was archived by the owner on Sep 12, 2026. It is now read-only.

Fix failing CI build workflow - #37

Merged
PromoFaux merged 2 commits into
masterfrom
copilot/fix-build-failures
Sep 2, 2026
Merged

PromoFaux merged 2 commits into
masterfrom
copilot/fix-build-failures

Conversation

Copilot AI commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

CI builds were failing on ubuntu-latest runners with mono: not found, exit code 127.

Root cause

The nuget restore $SOLUTION step used nuget.exe (installed via NuGet/setup-nuget), which requires Mono on Linux. GitHub-hosted ubuntu-latest runners no longer ship Mono by default.

Fix

All projects (Matterhook.NET.MatterhookClient, Matterhook.NET.MatterhookClient.Tests, ManualTests) are SDK-style with PackageReference (no packages.config), so Mono/nuget.exe isn't required at all:

  • Removed the NuGet/setup-nuget step and nuget restore; replaced with dotnet restore $SOLUTION
  • Replaced nuget push with dotnet nuget push in the Publish step
  • Bumped actions/checkout@v2v4 and actions/setup-dotnet@v1v4 to drop deprecated Node 20 runtime warnings

Not included

Did not add a dependabot.yml config, since another agent is reportedly already working on that separately.

Co-authored-by: PromoFaux <1998970+PromoFaux@users.noreply.github.com>
@PromoFaux

Copy link
Copy Markdown
Owner

@copilot please comment out the part that uploads to nuget for the time being - I need to dig about and find my account details! (Or maybe just use github to host it.. not sure how that works)

Co-authored-by: PromoFaux <1998970+PromoFaux@users.noreply.github.com>

Copilot AI commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please comment out the part that uploads to nuget for the time being - I need to dig about and find my account details! (Or maybe just use github to host it.. not sure how that works)

Commented out the Publish step in 9a2f930 - CI will now just build and test on PRs/tags, no package pushes until you're ready.

@PromoFaux
PromoFaux merged commit 0a92b51 into master Sep 2, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants