Skip to content

Conversation

@KSemenenko
Copy link
Member

Summary

  • update the CI workflow to restore, build, test, and format against the Presidio.slnx solution
  • update the release workflow to pack and publish using Presidio.slnx so GitHub Actions no longer looks for the missing Presidio.sln file

Testing

  • dotnet format Presidio.slnx --verify-no-changes (fails: existing xUnit1039 analyzer errors in analyzer test data definitions)
  • dotnet test Presidio.slnx --configuration Release (fails: existing xUnit1039 analyzer errors in analyzer test data definitions)

Codex Task

Copilot AI review requested due to automatic review settings November 7, 2025 21:46
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the GitHub Actions workflow files to use the new .slnx solution file format instead of the legacy .sln format. The repository has already migrated to using Presidio.slnx (an XML-based solution format introduced in Visual Studio 2022 version 17.10), and this change updates the CI/CD pipelines to reference the correct file.

  • Updates all dotnet CLI commands in workflows to reference Presidio.slnx
  • Ensures consistency between the repository structure and the build automation

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/ci.yml Updates all dotnet commands (restore, format, build, test) to use Presidio.slnx instead of Presidio.sln
.github/workflows/release.yml Updates all dotnet commands (restore, build, test, pack) to use Presidio.slnx instead of Presidio.sln

@KSemenenko KSemenenko merged commit 541504b into main Nov 8, 2025
1 of 2 checks passed
@KSemenenko KSemenenko deleted the codex/install-dotnet-9-and-fix-ci branch November 8, 2025 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants