Skip to content

Commit

Permalink
Removing the default of "all" for RunbooksToClone and ChannelsToClone.
Browse files Browse the repository at this point in the history
  • Loading branch information
BobJWalker committed Nov 18, 2021
1 parent 2f3e85a commit cae125c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions CloneSpace.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,6 @@ $CloneProjectLogos = Test-OctopusTrueFalseParameter -parameterValue $CloneProjec
$CloneTenantLogos = Test-OctopusTrueFalseParameter -parameterValue $CloneTenantLogos -parameterName "CloneTenantLogos" -defaultValue $true
$CloneTenantVariables = Test-OctopusTrueFalseParameter -parameterValue $CloneTenantVariables -parameterName "CloneTenantVariables" -defaultValue $false

$RunbooksToClone = Test-OctopusNewListParameter -parameterValue $RunbooksToClone -parameterName "RunbooksToClone"
$ChannelsToClone = Test-OctopusNewListParameter -parameterValue $ChannelsToClone -parameterName "ChannelsToClone"

if ($null -ne $CertificatesToClone -and $CertificatesToClone.ToLower().Trim() -eq "all")
{
Write-OctopusCritical "The parameter CertificatesToClone is set to 'all'. That is the one parameter that cannot be set to all. You must specify specific certificates to clone with their password."
Expand Down
2 changes: 1 addition & 1 deletion src/Core/ParameterVerification.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ function Test-OctopusNewListParameter

if ([string]::IsNullOrWhiteSpace($parameterValue))
{
Write-OctopusWarning "The paramter $parameterName is empty or null but this is a paramter you previously didn't have to set. Setting to 'all' so it doesn't break your existing clone."
Write-OctopusWarning "The parameter $parameterName is empty or null but this is a parameter you previously didn't have to set. Setting to 'all' so it doesn't break your existing clone."
return "all"
}

Expand Down

0 comments on commit cae125c

Please sign in to comment.