Skip to content

Commit 21ba6c7

Browse files
authored
Merge pull request #93 from PlagueHO/Issue-84
[Techdebt] Fix all PS Script Analyzer warnings - Fixes #84
2 parents a7954f5 + 6ded878 commit 21ba6c7

8 files changed

Lines changed: 481 additions & 365 deletions

File tree

DSCResources/CommonResourceHelper.psm1

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,12 +48,12 @@ function Get-LocalizedData
4848
#>
4949
function Remove-CommonParameter
5050
{
51-
[OutputType([hashtable])]
51+
[OutputType([System.Collections.Hashtable])]
5252
[cmdletbinding()]
5353
param
5454
(
5555
[Parameter(Mandatory = $true)]
56-
[hashtable]
56+
[System.Collections.Hashtable]
5757
$Hashtable
5858
)
5959

@@ -88,16 +88,18 @@ function Test-DscParameterState
8888
param
8989
(
9090
[Parameter(Mandatory = $true)]
91-
[hashtable]
91+
[System.Collections.Hashtable]
9292
$CurrentValues,
9393

9494
[Parameter(Mandatory = $true)]
9595
[object]
9696
$DesiredValues,
9797

98+
[Parameter()]
9899
[string[]]
99100
$ValuesToCheck,
100101

102+
[Parameter()]
101103
[switch]
102104
$TurnOffTypeChecking
103105
)
@@ -377,10 +379,12 @@ function New-InvalidOperationException
377379
[CmdletBinding()]
378380
param
379381
(
382+
[Parameter()]
380383
[ValidateNotNullOrEmpty()]
381384
[String]
382385
$Message,
383386

387+
[Parameter()]
384388
[ValidateNotNull()]
385389
[System.Management.Automation.ErrorRecord]
386390
$ErrorRecord

0 commit comments

Comments
 (0)