Skip to content

Conversation

@mattleibow
Copy link
Member

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description of Change

Issues Fixed

Fixes #

Copilot AI review requested due to automatic review settings June 23, 2025 14:48
@mattleibow mattleibow requested a review from a team as a code owner June 23, 2025 14:48
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 Copilot instructions by adding a new section that enforces including a testing note at the top of every PR description.

  • Introduces an “Opening PRs” section in .github/copilot-instructions.md
  • Provides a required snippet for the PR description to guide users to test PR builds
  • Clarifies placement of the testing note in PR descriptions
Comments suppressed due to low confidence (1)

.github/copilot-instructions.md:174

  • [nitpick] The instruction 'without the block quotes' may be confusing since the snippet itself uses blockquote syntax. Consider rephrasing to clearly explain which parts of the snippet should be included in the PR description and what should be omitted.
Always put that at the top, without the block quotes. Without it, the users will NOT be able to try the PR and your work will have been in vain!

@mattleibow mattleibow merged commit 7cc0f15 into main Jun 23, 2025
4 checks passed
@mattleibow mattleibow deleted the mattleibow-patch-1 branch June 23, 2025 14:59
NanthiniMahalingam pushed a commit to NanthiniMahalingam/maui that referenced this pull request Jun 27, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants