Skip to content

.NET 10 Migration + Version Comparison Fix#630

Merged
rygel merged 5 commits into
developfrom
feat/cycle-14-core
Jun 11, 2026
Merged

.NET 10 Migration + Version Comparison Fix#630
rygel merged 5 commits into
developfrom
feat/cycle-14-core

Conversation

@rygel

@rygel rygel commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Changes

  • task-45: Fix \IsNewerVersion\ for -develop\ suffix in prerelease tags

    • \ParseAppVersion\ now strips non-numeric suffixes (e.g. -develop) from beta number before parsing
    • \2.3.6-beta.2-develop\ now correctly compares as beta.2 instead of defaulting to 0
    • 4 new test cases covering develop-suffix scenarios

  • task-47: Migrate entire solution from .NET 8.0 to .NET 10.0

    • TargetFramework updated in all 10 .csproj files
    • Added \global.json\ pinning SDK to 10.0.301 with latestFeature rollForward
    • CI/CD workflows updated: dotnet-version 8.0.x to 10.0.x (9 workflows + composite action)
    • Test artifact paths updated in tests.yml

Test Results

  • 1365 passed (AIUsageTracker.Tests) on net10.0
  • 160 passed (Monitor.Tests) on net10.0
  • 0 failures

@actions-user
Fix IsNewerVersion for -develop suffix in prerelease tags
538f72a 
ParseAppVersion now strips non-numeric suffixes (e.g. -develop) from
the beta number before parsing, so 2.3.6-beta.2-develop correctly
compares as beta.2 instead of failing to parse and defaulting to 0.

Adds test cases covering develop-suffix scenarios.
@actions-user
Migrate entire solution from .NET 8.0 to .NET 10.0
d2d2abe 
- Update TargetFramework in all 10 .csproj files (net8.0 -> net10.0,
  net8.0-windows10.0.17763.0 -> net10.0-windows10.0.17763.0)
- Add global.json to pin SDK to 10.0.301 with latestFeature rollForward
- Update CI/CD workflows: dotnet-version 8.0.x -> 10.0.x in all workflows
  (tests, publish, codeql, code-coverage, slim-screenshot-baseline,
  analyzer-quality-gate, provider-contract-drift, monitor-openapi-contract,
  update-screenshot-baselines)
- Update setup-dotnet-cache composite action default to 10.0.x
- Update test artifact paths in tests.yml (net8.0 -> net10.0)
- All 1525 tests pass (1365 + 160 Monitor.Tests)
- NuGet packages already on 10.0.x versions, no package updates needed
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Large PR Detected

This PR contains 1109 lines changed across 57 files.

Details:

  • 📁 Files changed: 57
  • ➕ Insertions: 497
  • ➖ Deletions: 612
  • 📊 Total: 1109 lines

Recommendation:
Consider splitting this PR into smaller, focused changes to make review easier and faster. PRs under 500 lines are ideal for thorough code review.

This is an automated message. Feel free to ignore if this PR is intentionally large.

@actions-user
Fix CI: update global.json SDK version and script paths for .NET 10
abc466f 
- global.json: change SDK from 10.0.301 to 10.0.100 with latestFeature
  rollForward so CI runners with 10.0.300 resolve correctly
- Update 7 PowerShell scripts that hardcoded net8.0 output paths to
  net10.0 (verify-monitor-openapi-contract, verify_slim_theme_smoke,
  generate_screenshots, generate_card_catalog, generate_web_screenshots,
  resolve-test-assembly, run-slim-and-monitor)
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Large PR Detected

This PR contains 1127 lines changed across 64 files.

Details:

  • 📁 Files changed: 64
  • ➕ Insertions: 506
  • ➖ Deletions: 621
  • 📊 Total: 1127 lines

Recommendation:
Consider splitting this PR into smaller, focused changes to make review easier and faster. PRs under 500 lines are ideal for thorough code review.

This is an automated message. Feel free to ignore if this PR is intentionally large.

@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Large PR Detected

This PR contains 1127 lines changed across 64 files.

Details:

  • 📁 Files changed: 64
  • ➕ Insertions: 506
  • ➖ Deletions: 621
  • 📊 Total: 1127 lines

Recommendation:
Consider splitting this PR into smaller, focused changes to make review easier and faster. PRs under 500 lines are ideal for thorough code review.

This is an automated message. Feel free to ignore if this PR is intentionally large.

@actions-user
Add /api/providers/{providerId}/test to openapi.yaml
5a94b60 
Pre-existing gap: the provider test endpoint was added in task-41 (C12)
but never documented in the OpenAPI contract. The contract drift check
has been failing since PR #628 merged.
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Large PR Detected

This PR contains 1145 lines changed across 65 files.

Details:

  • 📁 Files changed: 65
  • ➕ Insertions: 524
  • ➖ Deletions: 621
  • 📊 Total: 1145 lines

Recommendation:
Consider splitting this PR into smaller, focused changes to make review easier and faster. PRs under 500 lines are ideal for thorough code review.

This is an automated message. Feel free to ignore if this PR is intentionally large.

@rygel
rygel merged commit be4c67f into develop Jun 11, 2026
22 of 23 checks passed
@rygel
rygel deleted the feat/cycle-14-core branch June 11, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size: large

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rygel @actions-user