Skip to content

Conversation

@jfversluis
Copy link
Member

Adds some more verbose instructions, and commented out csproj nodes, for enabling the TargetFrameworks in the ManualTests app.

Copilot AI review requested due to automatic review settings September 8, 2025 15:42
@jfversluis jfversluis added area-testing Unit tests, device tests testing-manual labels Sep 8, 2025
@jfversluis jfversluis requested a review from a team as a code owner September 8, 2025 15:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the documentation and configuration options for the ManualTests app by providing clearer instructions for specifying target frameworks when using external NuGet packages instead of workloads.

  • Adds detailed instructions in README.md for manually configuring target frameworks
  • Provides commented-out MSBuild properties in the project file for easier configuration
  • Clarifies the relationship between MauiVersion and target framework selection

Reviewed Changes

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

File Description
src/Controls/tests/ManualTests/README.md Added detailed instructions for target framework configuration and usage examples
src/Controls/tests/ManualTests/Controls.ManualTests.csproj Added commented MSBuild property examples for platform configuration

jsuarezruiz
jsuarezruiz previously approved these changes Sep 8, 2025
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jfversluis jfversluis merged commit 0d465cf into main Sep 8, 2025
88 of 129 checks passed
@jfversluis jfversluis deleted the jfversluis-patch-7 branch September 8, 2025 20:21
@github-actions github-actions bot locked and limited conversation to collaborators Oct 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants