Fix crashing when PATH contains multiple choco installations #148
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.
When for some reason the path environment variable contains two installations of chocolatey this function will return a list, which causes a syntax error down the line. Since running choco.exe on the machine will also choose the first installation in the path, I think limiting the found installations to the first is the desired behavior here.
Description Of Changes
Restricted the output of Get-Command to the first finding
Motivation and Context
I encountered a crashing win_chocolatey task, when on the target machine the path variable contained multiple choco installations. Limiting the finding to the first in the PATH, the same way it would be resolved in the powershell on the machine, fixes this problem.
Testing
I have tried this out before and after the change, the change fixes this issue
Operating Systems Testing
Change Types Made
Change Checklist
Related Issue
Fixes #147