Skip to content

Fix project path and update Docker base image to .NET 10.0#70

Open
alexmanie wants to merge 3 commits intoAzure-Samples:mainfrom
alexmanie:main
Open

Fix project path and update Docker base image to .NET 10.0#70
alexmanie wants to merge 3 commits intoAzure-Samples:mainfrom
alexmanie:main

Conversation

@alexmanie
Copy link

Purpose

This pull request updates the development environment and corrects a project reference in the solution file.
These changes help ensure compatibility with newer tools and prevent build issues due to incorrect file paths.

Development environment update:

  • Updated the base image in .devcontainer/Dockerfile from the .NET 8.0 Bookworm image to the .NET 10.0 Noble image, ensuring the dev container uses the latest .NET platform and Ubuntu base.

Solution/project configuration:

  • Fixed the project file reference for WeatherApp in FunctionsMcpTool.sln to point to McpWeatherApp.csproj instead of the incorrect WeatherApp.csproj, resolving potential build or project loading errors.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code and open it in devcontainer or codespace. Manual run dotnet restore at root folder level.
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
dotnet restore

What to Check

Verify that the following are valid

  • NuGet restore works as expected

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.

1 participant