Skip to content

Conversation

mutn3ja
Copy link
Contributor

@mutn3ja mutn3ja commented Nov 8, 2016

  • Task is able to detect VS15 installation
  • Added tests

[Parameter(Mandatory = $true)]
[string]$LiteralPath)

Write-Host "Testing container: '$LiteralPath'"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

' is not required


Write-Host "Testing container: '$LiteralPath'"
if ((Test-Path -LiteralPath $LiteralPath -PathType Container)) {
Write-Host 'Exists.'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

meaningful message?

else # go for latest installation
{
# first find willow installation
$vs15 = Get-VisualStudio_15_0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pull this code into another function

@nigurr
Copy link

nigurr commented Nov 9, 2016

LGTM

@mutn3ja mutn3ja merged commit 9740b7a into master Nov 9, 2016
@bryanmacfarlane bryanmacfarlane deleted the users/prmutn/VSTestLocationWillow branch January 31, 2018 00:26
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

Successfully merging this pull request may close these issues.

3 participants