Skip to content

Fix README path reference from Demos to Samples#4070

Draft
Copilot wants to merge 2 commits intocrickman/dotnet-samples-portablefrom
copilot/sub-pr-4064
Draft

Fix README path reference from Demos to Samples#4070
Copilot wants to merge 2 commits intocrickman/dotnet-samples-portablefrom
copilot/sub-pr-4064

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

Motivation and Context

Addresses review feedback from PR #4064 (comment #4064 (comment)) regarding an outdated path reference that was missed during the Demos → Samples migration.

Description

Updated dotnet/src/Shared/Samples/README.md line 20 to reference the correct Samples folder path instead of the obsolete Demos folder:

- <Compile Include="$(MSBuildThisFileDirectory)\..\src\Shared\Demos\*.cs" LinkBase="" Visible="false" />
+ <Compile Include="$(MSBuildThisFileDirectory)\..\src\Shared\Samples\*.cs" LinkBase="" Visible="false" />

This ensures the documentation accurately reflects the current repository structure for users copying SampleEnvironment.cs to external projects.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: crickman <66376200+crickman@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Apply feedback from review on .NET Samples simplification PR Fix README path reference from Demos to Samples Feb 19, 2026
Copilot AI requested a review from crickman February 19, 2026 05:46
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

Comments