Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recommend parameter rename for VcList parameter in any Cmdlet that calls Test-VcListObject #155

Open
BronsonMagnan opened this issue Jan 12, 2024 · 0 comments
Assignees

Comments

@BronsonMagnan
Copy link

Expected Behavior

Because the parameter is named VcList, we expect to be able to supply the output of Get-VcList to the parameter

Current Behavior

The parameter requires the output object of 'Save-VCRedist', which is the same date with an additional property of Path included.

Failure Information (for bugs)

Cmdlets fail on private function Test-VcListObject validator.

Steps to Reproduce

Do what is intuitive

Context

In most PowerShell modules, the standard typically is a Parameter Name that matches the name of a defined Noun in the module, should accept that data type generated by the Get-Noun cmdlet if one is present.

Recommendations

Rename VcList parameter in cmdlets that want the input of Save-VCRedist.
Additionally implement a cmdlet that can generate this object array without having to re-download the redists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants