Repository navigation
Conversation
Add a -NoCheckout switch to Tools\PRTest.ps1 that fetches the PR's validated merged manifest from the WinGet Validator artifact instead of checking out the pull request locally. * Resolve the PR head SHA and locate the app-scoped 10. Validation Completed check run * Parse the artifact download URL from the check output and download, extract, and stage the singleton YAML manifest into a folder named for the PR number under $env:TEMP\WinGet-PRTest * Run SandboxTest.ps1 against the staged manifest with -SkipManifestValidation implied * Preserve existing checkout-mode behavior * Document exit codes for missing dependencies, checkout failure, missing validation check, missing artifact URL, and missing YAML
Owner
Author
|
Closing to re-target this change as a draft PR against upstream microsoft/winget-pkgs. |
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.
Summary
Adds a
-NoCheckoutswitch toTools\PRTest.ps1. Instead of checking out the pull request locally, it retrieves the pull request's WinGet Validator10. Validation Completedcheck run, downloads the associated validation artifact, extracts the singleton merged manifest YAML, stages it in a PR-numbered folder under$env:TEMP\WinGet-PRTest, and runsSandboxTest.ps1against it (implying-SkipManifestValidation).Existing checkout-mode behavior is unchanged.
Changes
Tools\PRTest.ps1:-NoCheckoutswitch10. Validation Completedcheck runArtifactDownloadUrlfrom the check output, downloads and extracts the artifact, stages the single YAML manifest into$env:TEMP\WinGet-PRTest\<PullRequest>Validation performed
SandboxTest.ps1stub (no installation launched): resolved the check run, downloaded and extracted the artifact, staged the single YAML manifest, and passedSkipManifestValidation = true10. Validation Completedcheck resolves correctly for New version: laurentiu021.SysManager version 1.113.14 microsoft/winget-pkgs#440812, New version: agentty.agentty version 0.9.10 microsoft/winget-pkgs#440811, and New version: demkada.ArgyCode version 2.7.3 microsoft/winget-pkgs#440810-NoCheckoutNote
No related open/closed issue was found in microsoft/winget-pkgs, and issue creation requires explicit confirmation per repository conventions that was not available during this session. This PR does not yet link to an issue — please link one or confirm creating a new
feature_request.ymlissue before merge.Generated with assistance from GitHub Copilot.
Checklist