Add dogfooding scripts and workflow for PR testing#1150
Open
marshalhayes wants to merge 6 commits intoCommunityToolkit:mainfrom
Open
Add dogfooding scripts and workflow for PR testing#1150marshalhayes wants to merge 6 commits intoCommunityToolkit:mainfrom
marshalhayes wants to merge 6 commits intoCommunityToolkit:mainfrom
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds “dogfooding” automation to help developers test NuGet packages produced by a pull request’s CI build artifacts, via a new PR-commenting workflow and companion install scripts.
Changes:
- Added a Bash script to download PR build artifacts (NuGet packages) and register a local NuGet source for testing.
- Added a PowerShell script providing the same workflow for Windows/PowerShell users.
- Added a GitHub Actions workflow that comments on PRs with copy/paste dogfooding commands.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
eng/scripts/dogfood-pr.sh |
Downloads nuget-packages artifact for a PR run and installs packages into a local “hive”, then configures a NuGet source. |
eng/scripts/dogfood-pr.ps1 |
PowerShell equivalent of the Bash dogfooding script (download, install, configure source, print undo steps). |
.github/workflows/dogfood-comment.yml |
Posts/updates a PR comment with instructions to run the dogfooding scripts. |
You can also share your feedback on Copilot code review. Take the survey.
Use full quoted paths instead of unquoted display paths in the printed undo commands so they work correctly when copy-pasted on systems where paths contain spaces. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add guards for --run-id and --install-path to check that a value is provided and that it doesn't look like another flag. Prevents unbound variable errors and silent flag consumption. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add unzip to prerequisite checks - Eliminate standalone jq dependency by using gh api --jq directly - Replace GNU-specific grep -oP with portable sed for nuspec parsing - Replace GNU-specific find -printf with portable find -exec basename - Simplify configure_nuget_source() by dropping --configfile and letting dotnet nuget use default config resolution Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace bash-style 'rm -rf', '> $null', and '&&' with PowerShell equivalents: Remove-Item -Recurse -Force, Out-Null, and semicolons. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep add/update behavior on dotnet's default NuGet config resolution, but restore --configfile in the printed undo commands when the resolved config path is known so users can undo against the same nuget.config from any working directory. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
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 PR introduces a new automated workflow and two scripts to make it easier for developers to "dogfood" NuGet packages built from a pull request.
The workflow automatically posts instructions as a comment on new PRs, and the scripts allow users to quickly download and install packages from PR build artifacts using either Bash or PowerShell.
PR Checklist
Other information