Test - don't merge

[Yuri05]

Fixes #

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Type of change

Please mark relevant options with an x in the brackets.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires documentation changes (link at least one user or developer documentation issue):
• Algorithm update - updates algorithm documentation/questions/answers etc.
• Other (please describe):

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Integration tests
• Unit tests
• Manual tests
• No tests required

Reviewer checklist

Mark everything that needs to be checked before merging the PR.

• Check if the code is well documented
• Check if the behavior is what is expected
• Check if the code is well tested
• Check if the code is readable and well formatted
• Additional checks (document below if any)
• Check if documentation update issue(s) are created if the option This change requires a documentation update above is selected

Screenshots (if appropriate):

Questions (if appropriate):

[Copilot]

Pull request overview

This PR updates the PKSim project from version 12.2 to 12.3, including package references, workflow configurations, and project metadata. The PR title indicates this is a test change not intended for merging.

Changes:

• Version numbers updated from 12.2.0 to 12.3.0 across all project files
• OSPSuite package references updated from 12.2.26 to 12.3.1
• GitHub Actions workflow files renamed and updated to reference version 12.3
• Build and test workflows consolidated and simplified

Reviewed changes

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

Show a summary per file

File	Description
tests/PKSim.UI.Tests/PKSim.UI.Tests.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
tests/PKSim.Tests/PKSim.Tests.csproj	Updated version to 12.3.0 and OSPSuite.Core to 12.3.1
tests/PKSim.R.Tests/PKSim.R.Tests.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSimRDependencyResolution/PKSimRDependencyResolution.csproj	Updated version to 12.3.0 and OSPSuite.R to 12.3.1
src/PKSim/PKSim.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.UI/PKSim.UI.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.UI.Starter/PKSim.UI.Starter.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.R/PKSim.R.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.Presentation/PKSim.Presentation.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.Infrastructure/PKSim.Infrastructure.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.Core/PKSim.Core.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.CLI/PKSim.CLI.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.CLI.Core/PKSim.CLI.Core.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.BatchTool/PKSim.BatchTool.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.Assets/PKSim.Assets.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
src/PKSim.Assets.Images/PKSim.Assets.Images.csproj	Updated version to 12.3.0 and OSPSuite packages to 12.3.1
rakefile.rb	Updated application directory path from '12.2' to '12.3'
README.md	Updated workflow reference from build-nightly_12.2.yml to build-nightly_12.3.yml
PKSim.sln	Updated Visual Studio version and workflow file references
.github/workflows/nightly-badge.yml	Updated workflow name references from 12.2 to 12.3
.github/workflows/coverage.yml	Updated build version from 12.2.9999 to 12.3.9999
.github/workflows/build-nightly_12.3.yml	Renamed from 12.2 version, updated version references and simplified build commands
.github/workflows/build-and-test_12.2.yml	Removed (consolidated into build-and-test.yml)
.github/workflows/build-and-test.yml	New consolidated workflow for version 12.3
.github/workflows/build-and-test-pr_12.2.yml	Removed (consolidated into build-and-test.yml)

Comments suppressed due to low confidence (1)

.github/workflows/build-nightly_12.3.yml:120

• The if: always() condition on the test log upload step is a good practice to ensure logs are uploaded even when tests fail. This change improves debugging capabilities.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[greptile-apps]

Greptile Summary

This PR upgrades the project from version 12.2 to 12.3 across all configuration files, workflows, and project dependencies.

Key Changes:

• Deleted old 12.2-specific workflows (build-and-test-pr_12.2.yml, build-and-test_12.2.yml) and replaced with simplified build-and-test.yml that uses centralized test-csharp.yml workflow from upstream Workflows repository
• Renamed build-nightly_12.2.yml to build-nightly_12.3.yml with improvements: removed hardcoded MAJOR, MINOR, RUN environment variables, simplified build/test commands, and added if: always() to test log upload step for better failure handling
• Updated all 17 .csproj files to use version 12.3.0 and upgraded all OSPSuite package dependencies from 12.2.26 to 12.3.1
• Updated Visual Studio solution file to reference VS version 18.1

Note: The PR title states "Test - don't merge" indicating this is intended for testing purposes only and should not be merged to the base branch.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk (if intended for merging)
• All changes are systematic version upgrades following consistent patterns. The workflow simplification uses established upstream workflows. The addition of if: always() to test log uploads is a best practice improvement. No functional logic changes, security issues, or breaking changes detected.
• No files require special attention

Important Files Changed

Filename	Overview
.github/workflows/build-and-test.yml	Simplified workflow to use centralized test-csharp.yml from Workflows repository, updated to version 12.3.9999
.github/workflows/build-nightly_12.3.yml	Renamed from 12.2 to 12.3, removed hardcoded env variables, simplified build commands, added if: always() to test log upload
src/PKSim.Core/PKSim.Core.csproj	Updated default version to 12.3.0 and OSPSuite packages from 12.2.26 to 12.3.1
src/PKSim.Infrastructure/PKSim.Infrastructure.csproj	Updated default version to 12.3.0 and all OSPSuite packages from 12.2.26 to 12.3.1

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant PR as Pull Request #3377
    participant GHA as GitHub Actions
    participant Workflows as OSP Workflows Repo
    participant Deps as Package Dependencies
    
    Dev->>PR: Push version upgrade changes (12.2 → 12.3)
    
    Note over PR: Update workflow files
    PR->>GHA: Delete old 12.2 workflows
    PR->>GHA: Create simplified build-and-test.yml
    PR->>GHA: Rename/update build-nightly to 12.3
    PR->>GHA: Update coverage.yml and nightly-badge.yml
    
    Note over PR: Update project files (25 files)
    PR->>Deps: Update default version to 12.3.0
    PR->>Deps: Upgrade OSPSuite packages (12.2.26 → 12.3.1)
    
    Note over PR: Update configuration
    PR->>PR: Update rakefile.rb application path
    PR->>PR: Update README.md badge link
    PR->>PR: Update PKSim.sln VS version
    
    alt On Pull Request or Push to develop
        GHA->>Workflows: Call test-csharp.yml@main
        Workflows->>GHA: Execute standardized build & test
        GHA->>GHA: Run tests with version 12.3.9999
    end
    
    alt On Nightly Build (if recent commits)
        GHA->>GHA: Check latest commit timespan
        GHA->>GHA: Build with version 12.3.{run_number}
        GHA->>GHA: Run tests (upload logs with if: always())
        GHA->>GHA: Sign executables
        GHA->>GHA: Create and upload artifacts
    end

Loading

[greptile-apps]

Greptile found no issues!

From now on, if a review finishes and we haven't found any issues, we will not post anything, but you can confirm that we reviewed your changes in the status check section.

_{This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR".}

[msevestre]

This graph lol 😝