| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Note
Copilot was unable to run its full agentic suite in this review.
Adds explicit handling for resource get when no --input/stdin is provided, preventing PowerShell parameter-binding errors and returning a clean non-zero exit code.
Changes:
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| test/DscResource/PSResourceGetDSCResource.Tests.ps1 | Adds a regression test for resource get without input to ensure clean failure behavior. |
| src/dsc/psresourceget.ps1 | Adds a guard clause to reject missing stdin/--input before attempting JSON parsing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
PR Summary
This pull request adds a simple input validation when it's empty for the GetOperation on the Repository resource.
PR Context
When --input is omitted, DSC closes the child process stdin without writing anything. The
resource manifest uses "input": "stdin" and invokes the script as $Input | ./psresourceget.ps1 ..., so $Input is an empty enumerator and the $stdinput pipeline parameter is never bound (keeping it $null).
Fixes #2001
PR Checklist