Skip to content

Update Dockerfiles to use .NET SDK and ASP.NET Core 10.0#176

Merged
frasermclean merged 2 commits into
mainfrom
fix/dockerfile-net10
Feb 2, 2026
Merged

Update Dockerfiles to use .NET SDK and ASP.NET Core 10.0#176
frasermclean merged 2 commits into
mainfrom
fix/dockerfile-net10

Conversation

@frasermclean

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings February 2, 2026 04:03
@github-actions

github-actions Bot commented Feb 2, 2026

Copy link
Copy Markdown

Test results

50 tests  ±0   50 ✅ ±0   2s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 3 files   ±0    0 ❌ ±0 

Results for commit 7559d43. ± Comparison against base commit 79913e8.

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.

Pull request overview

This pull request updates the Dockerfiles to use .NET SDK and ASP.NET Core 10.0, aligning the container images with the project's target framework which already specifies net10.0. The update affects both the API and Jobs services.

Changes:

  • Updated .NET SDK version from 9.0 to 10.0 in build stages
  • Updated ASP.NET Core runtime from 9.0 to 10.0 for the API service
  • Updated Azure Functions runtime from dotnet-isolated9.0 to dotnet-isolated10.0 for the Jobs service
  • Fixed OCI label format by adding missing equals sign in the Jobs Dockerfile

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
backend/src/Jobs/Dockerfile Updated SDK to 10.0, Azure Functions runtime to dotnet-isolated10.0, and fixed OCI label format
backend/src/Api/Dockerfile Updated SDK to 10.0 and ASP.NET Core runtime to 10.0

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

Comment thread backend/src/Jobs/Dockerfile Outdated
@codecov

codecov Bot commented Feb 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.47%. Comparing base (79913e8) to head (d9d7c55).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #176   +/-   ##
=======================================
  Coverage   78.47%   78.47%           
=======================================
  Files         112      112           
  Lines        1700     1700           
  Branches       68       68           
=======================================
  Hits         1334     1334           
  Misses        350      350           
  Partials       16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@frasermclean frasermclean merged commit e1f49a9 into main Feb 2, 2026
1 check passed
@frasermclean frasermclean deleted the fix/dockerfile-net10 branch February 2, 2026 04:07
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.

2 participants