Skip to content

Modernise solution tooling and .NET 10 CI coverage - #254

Merged
frasermolyneux merged 3 commits into
mainfrom
copilot/modernise-dotnet-solution
Aug 31, 2026
Merged

frasermolyneux merged 3 commits into
mainfrom
copilot/modernise-dotnet-solution

Conversation

Copilot AI commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Migrates the repository to the modern .slnx solution format while retaining intentional net9.0;net10.0 multi-targeting. CI now explicitly executes runtime integration coverage alongside existing restore, build, package, and unit-test paths.

  • Solution migration

    • Replaced MX.Api.Abstractions.sln with MX.Api.Abstractions.slnx.
    • Preserved all package, unit-test, and integration-test projects.
  • Tooling and documentation

    • Updated VS Code tasks, contributor guidance, Copilot instructions, and maintenance documentation to use .slnx.
  • Runtime CI coverage

    • Added integration-test execution to feature, PR, and release validation workflows.
    • Exercises startup, service registration, client configuration, serialization, and HTTP integration paths for both target frameworks.
- name: Run integration tests
  run: dotnet test src/MX.Api.IntegrationTests/MX.Api.IntegrationTests.csproj --configuration Release --no-build
  • Project compatibility
    • Confirmed all projects already use SDK-style project files.
    • Retained existing package generation and net9.0;net10.0 support.

Co-authored-by: frasermolyneux <34033625+frasermolyneux@users.noreply.github.com>
Copilot AI changed the title [WIP] Modernise .NET solution and target framework Modernise solution tooling and .NET 10 CI coverage Aug 31, 2026
Copilot AI requested a review from frasermolyneux August 31, 2026 20:10
@frasermolyneux
frasermolyneux marked this pull request as ready for review August 31, 2026 20:18
Copilot AI lite review requested due to automatic review settings August 31, 2026 20:18
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new MX.Api.Abstractions.slnx file appears to use an invalid schema (platforms placed under configurations and missing Debug/Release configurations), which is likely to break solution parsing/builds.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR modernizes the repository’s solution tooling by migrating from the legacy .sln format to the newer .slnx solution format, and extends CI validation to explicitly run the runtime integration test suite alongside the existing dotnet-ci checks—supporting the repo’s net9.0;net10.0 multi-targeting goals from Issue #253.

Changes:

  • Added src/MX.Api.Abstractions.slnx and removed the legacy src/MX.Api.Abstractions.sln.
  • Updated contributor/dev tooling references (docs, VS Code tasks, Copilot instructions) to use the .slnx solution.
  • Updated feature/PR/release workflows to run MX.Api.IntegrationTests as part of CI validation.
File summaries
File Description
src/MX.Api.Abstractions.slnx Introduces the new XML-based solution file used by tooling/CI (currently has a schema issue noted in comments).
src/MX.Api.Abstractions.sln Removes the legacy Visual Studio solution file.
docs/package-maintenance.md Updates maintenance automation docs to use the .slnx solution path.
docs/development-workflows.md Updates workflow documentation and standard dev commands to reference .slnx and integration CI coverage.
AGENTS.md Updates repo agent guidance to reference .slnx in locations and commands.
.vscode/tasks.json Updates VS Code tasks to target the .slnx solution.
.github/workflows/build-and-test.yml Adds an integration test run step to feature branch CI.
.github/workflows/pr-verify.yml Adds an integration test run step to PR validation CI.
.github/workflows/release-version-and-tag.yml Adds an integration test run step to release validation CI.
.github/copilot-instructions.md Updates Copilot repo instructions to reference .slnx in validation commands.
Review details
  • Files reviewed: 10/10 changed files
  • Comments generated: 1
  • Review effort level: Lite

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

Comment thread src/MX.Api.Abstractions.slnx Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

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.

Modernise .NET solution and target framework

3 participants