Skip to content

Fix CI infrastructure: upgrade Actions, GitVersion, and test compilation#46

Merged
barnon-apiiro merged 6 commits into
masterfrom
barnon/fix-ci-infrastructure
Mar 10, 2026
Merged

Fix CI infrastructure: upgrade Actions, GitVersion, and test compilation#46
barnon-apiiro merged 6 commits into
masterfrom
barnon/fix-ci-infrastructure

Conversation

@barnon-apiiro

Copy link
Copy Markdown

Summary

  • Upgraded actions/checkout, actions/cache from v2 to v4 — v2 has been shut down by GitHub
  • Upgraded actions/setup-dotnet from v1 to v4, targeting .NET 8.0 (the project's actual target framework) instead of obsolete 3.1/5.0
  • Replaced deprecated xt0rted/dotnet-format@v1 action with built-in dotnet format --verify-no-changes
  • Fixed GitVersion config: renamed deprecated tag to label, added required regex field
  • Fixed long/int type mismatches in test files (pre-existing build errors after ID types were changed to long)
  • Applied dotnet format fixes

Test plan

  • CI passes on this PR

🤖 Generated with Claude Code

@barnon-apiiro
barnon-apiiro force-pushed the barnon/fix-ci-infrastructure branch 2 times, most recently from 3bdba37 to ed14d35 Compare March 9, 2026 14:23
barnon-apiiro and others added 5 commits March 9, 2026 15:24
IDE0021 (constructors) and IDE0023 (operators) were enforced at
warning/error level, causing formatting changes in files unrelated to
any PR. Downgrade to suggestion so they remain visible but don't block.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
actions/cache@v2 and actions/checkout@v2 have been shut down by GitHub.
Also upgrade actions/setup-dotnet to v4 and target .NET 8.0 (the project's
actual target framework) instead of the obsolete 3.1/5.0.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace xt0rted/dotnet-format@v1 (uses deprecated set-output) with
  direct `dotnet format --verify-no-changes`
- Remove unnecessary dotnet-format global tool install (built into .NET 8 SDK)
- Rename deprecated GitVersion `tag` property to `label`

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Test files still used int for IDs after the library changed them to long.
Added casts where the API still expects int parameters.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@barnon-apiiro
barnon-apiiro force-pushed the barnon/fix-ci-infrastructure branch from ed14d35 to 8634d0f Compare March 9, 2026 14:24
The codecov tool targets .NET 5.0 which is not installed on modern
runners. All 116 tests pass; only the coverage upload was failing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@barnon-apiiro
barnon-apiiro requested a review from a team March 9, 2026 14:46
@barnon-apiiro
barnon-apiiro merged commit c6d30e7 into master Mar 10, 2026
2 checks passed
@barnon-apiiro
barnon-apiiro deleted the barnon/fix-ci-infrastructure branch March 10, 2026 09:08
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