Upgrade assets-svc from .NET 8 to .NET 10 (LTS) - #34
Merged
Conversation
Move the modern assets-svc service to the current .NET 10 LTS across the app baseline. There is no pedagogical reason for .NET 8 (only the Java audit-svc is intentionally dated as the Module 6 modernization target). - TargetFramework net8.0 -> net10.0 (AssetsService.csproj + Tests csproj) - NuGet (reconciles Dependabot #30): Dapper 2.1.79, Microsoft.Data.Sqlite 10.0.11, Swashbuckle.AspNetCore 10.2.3; test deps to net10-compatible stable (Mvc.Testing 10.0.11, Test.Sdk 18.9.0, xunit 2.9.3, runner.visualstudio 3.1.5) - devcontainer dotnet feature 8.0 -> 10.0 (lock unchanged: feature installer is SDK-channel-agnostic) - assets-svc Dockerfile sdk/aspnet 8.0 -> 10.0 (net10 runtime) - Docs: README, CONTRIBUTING, assets-svc/README, PR + issue templates .NET 8 -> .NET 10 Verified on net10: build + tests green, service starts on :5001 (Swagger + /assets 200). Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part A — app baseline .NET 8 → .NET 10
Moves the modern
assets-svcservice to the current .NET 10 LTS. There is no pedagogical reason for .NET 8 — only the Javaaudit-svcis intentionally dated (the Module 6 modernization target). Java services are untouched.Changes
net8.0→net10.0inAssetsService.csprojandTests/AssetsService.Tests.csproj.Dapper2.1.35 → 2.1.79Microsoft.Data.Sqlite8.0.4 → 10.0.11Swashbuckle.AspNetCore6.6.2 → 10.2.3Microsoft.AspNetCore.Mvc.Testing→ 10.0.11,Microsoft.NET.Test.Sdk→ 18.9.0,xunit→ 2.9.3,xunit.runner.visualstudio→ 3.1.5dotnetfeature8.0→10.0.devcontainer-lock.jsonregenerated viadevcontainer upgrade— no change (the lock pins the feature installer, which is SDK-channel-agnostic and already resolves .NET 10).dotnet/sdk:8.0+dotnet/aspnet:8.0→:10.0(functional: a net10 assembly won't run on the aspnet:8.0 runtime). Authored here, not by merging Dependabot Bump the docker-base-images group across 7 directories with 6 updates #29.README.md,CONTRIBUTING.md,services/assets-svc/README.md, PR template, and both issue templates.NET 8→.NET 10.Verification (net10; host dotnet 10.0.203)
dotnet build— ✅ green (both projects targetnet10.0).dotnet test— ✅ green (2/2). Note:mainonly carries 2 test files; the "28 tests" figure is the post-delta learner-branch state and is validated in Part B.dotnet runon :5001 serves/swagger/index.html(200),/swagger/v1/swagger.json(200),/assets(200); SQLite schema/seed init OK.Scope / constraints
course-build/changes, no course workflow files.Note
Part B (learner-branch re-baseline) is planned separately and not in this PR. Because learner branches derive from the pinned tooling-free
acc-base(notmain), merging this does not move them. Part B re-baselines onto a new tooling-free base and re-derives the deltas (delta_01 + delta_05 hard-conflict on the net8 context). See the session plan; nothing there executes until this merges and Part B is approved.