Add PR dogfooding script + automatic comment #33043
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jfversluis
added
the
area-infrastructure
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces automated dogfooding infrastructure to help community members test PR builds before they're merged. The automation adds instructional comments to PRs, and provides comprehensive scripts for applying PR artifacts to local projects.
Key Changes
- Automated GitHub Actions workflow that posts dogfooding instructions on every PR
- Cross-platform scripts (Bash and PowerShell) for downloading and applying PR NuGet packages
- Comprehensive documentation explaining usage, requirements, and troubleshooting
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/dogfood-comment.yml |
New workflow that automatically posts/updates dogfooding instructions on PRs |
eng/scripts/get-maui-pr.sh |
Bash script for Unix systems to download and apply PR artifacts |
eng/scripts/get-maui-pr.ps1 |
PowerShell script for cross-platform PR artifact application |
eng/scripts/README.md |
Documentation for the dogfooding scripts with usage examples and troubleshooting |
.github/PULL_REQUEST_TEMPLATE.md |
Removes manual dogfooding note (now automated by workflow) |
StephaneDelcroix
approved these changes
Dec 8, 2025
| Write-Host "⚠️ $Message" -ForegroundColor Yellow | ||
| } | ||
|
|
||
| function Write-Error { |
There was a problem hiding this comment.
Write-Error is a built-in PowerShell cmdlet and should not be redefined. This function shadows the built-in cmdlet, which could cause confusion and unexpected behavior. Consider renaming this function to Write-ErrorMessage or similar to avoid conflicts.
Suggested change
| function Write-Error { | |
| function Write-ErrorMessage { |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces an automated workflow to add dogfooding instructions to pull requests, and provides comprehensive documentation for scripts that help users test PR builds locally. It also cleans up the PR template by removing a manual note about testing artifacts.
Dogfooding automation:
.github/workflows/dogfood-comment.yml) that automatically comments on PRs with instructions for testing the PR's build artifacts, updating the comment if one already exists. This workflow is triggered on PR events and can also be run manually.Documentation improvements:
README.mdineng/scriptsexplaining how to use theget-maui-pr.shandget-maui-pr.ps1scripts to fetch and apply PR NuGet artifacts, including usage examples, requirements, safety features, troubleshooting, and revert instructions.Template cleanup:
.github/PULL_REQUEST_TEMPLATE.md) that previously asked users to test PR artifacts and report results, as this is now automated.