PSSA SingularNouns and ShouldProcess

Active Directory/AD Groups/New-ADGroupsFromCsv.ps1

@@ -22,6 +22,8 @@
     Version: 0.1.0
     #>
     [CmdletBinding()]
+    [Diagnostics.CodeAnalysis.SuppressMessageAttribute('SupportsShouldProcess', '')]
+    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
     param (
         # CSV file with group names and descriptions (validate path)
         [Parameter(Mandatory, Position = 0)]
@@ -69,6 +71,7 @@
                 GroupCategory  = $group.Category
                 GroupScope     = $group.Scope
                 Description    = $group.Description
+                Server         = $Server
             }
             try {
                 New-ADGroup @GroupParams

Active Directory/AD Users/Get-ADInactiveUsers.ps1

@@ -3,6 +3,7 @@
     # DRAFT, WORK IN PROGRESS
 
     [CmdletBinding()]
+    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
     param (
         # Days to consider an account inactive
         [Parameter()]

Active Directory/AD Users/Get-AccountsWithPasswordNeverExpires.ps1

@@ -10,6 +10,7 @@
     Get-AccountsWithPasswordNeverExpires
     #>
     [CmdletBinding()]
+    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
     param ()
 
     Search-ADAccount -PasswordNeverExpires

Active Directory/AD Users/Search-KerbDelegatedAccounts.ps1

@@ -37,6 +37,7 @@
 #>
 
 [CmdletBinding()]
+[System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
 Param
 (
     # start the search at this DN. Default is to search all of the domain.

Active Directory/AD Users/Test-IsMemberOfProtectedUsers.ps1

@@ -38,6 +38,7 @@ function Test-IsMemberOfProtectedUsers {
     #>
 
     [CmdletBinding()]
+    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
     [OutputType([System.Boolean])]
     param (
         # User parameter accepts any input that is valid for Get-ADUser

Active Directory/Domain Services/Get-ADSitesWithoutLinks.ps1 → Active Directory/Domain Services/Get-ADSitesWithoutLink.ps1

@@ -1,4 +1,4 @@
-function Get-ADSitesWithoutLinks {
+function Get-ADSitesWithoutLink {
     <#
     .SYNOPSIS
     Get all Active Directory sites that are not included in any site link.

Active Directory/Get-AllADSIDHistoryDetails.ps1 → Active Directory/Get-AllADSIDHistoryDetail.ps1

@@ -1,4 +1,4 @@
-function Get-AllADSIDHistoryDetails {
+function Get-AllAdSidHistoryDetail {
     [CmdletBinding()]
     [OutputType([hashtable])]
     param ()

Active Directory/Get-AllADSIDHistorySourceDomains.ps1 → Active Directory/Get-AllADSIDHistorySourceDomain.ps1

@@ -1,4 +1,4 @@
-function Get-AllADSIDHistorySourceDomains {
+function Get-AllAdSidHistorySourceDomain {
     <#
     .SYNOPSIS
     Get a list of source domains from all Active Directory objects that have SID history.

Active Directory/Get-GPMissingPermissionGPOs.ps1 → Active Directory/Get-GPMissingPermissionGPO.ps1

@@ -1,44 +1,44 @@
 #Find Group Policies with Missing Permissions
-Function Get-GPOsMissingPermissions {
+Function Get-GPOsMissingPermission {
+    [CmdletBinding()]
+    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
 
-    $GPOs = Get-GPO -all
+    $GPOs = Get-GPO -All
 
     # Check for GPOs missing Authenticated Users and Domain Computers
     $GPOsMissingPermissions = New-Object System.Collections.ArrayList
     foreach ($item in $GPOs) {
-        $GPOPermAuthUsers       = Get-GPPermission -Guid $GPO.Id -All | Select-Object -ExpandProperty Trustee | Where-Object {$_.Name -eq "Authenticated Users"}
-        $GPOPermDomainComputers = Get-GPPermission -Guid $GPO.Id -All | Select-Object -ExpandProperty Trustee | Where-Object {$_.Name -eq "Domain Computers"}
+        $GPOPermAuthUsers = Get-GPPermission -Guid $GPO.Id -All | Select-Object -ExpandProperty Trustee | Where-Object { $_.Name -eq 'Authenticated Users' }
+        $GPOPermDomainComputers = Get-GPPermission -Guid $GPO.Id -All | Select-Object -ExpandProperty Trustee | Where-Object { $_.Name -eq 'Domain Computers' }
 
-        If (!$GPOPermAuthUsers -and !$GPOPermDomainComputers) { 
-            $GPOsMissingPermissions.Add($item)| Out-Null
+        If (!$GPOPermAuthUsers -and !$GPOPermDomainComputers) {
+            $GPOsMissingPermissions.Add($item) | Out-Null
         }
     }
     If ($GPOsMissingPermissions.Count -ne 0) {
-        Write-Warning  "The following Group Policy Objects do not grant any permissions to the 'Authenticated Users' or 'Domain Computers' groups:"
+        Write-Warning "The following Group Policy Objects do not grant any permissions to the 'Authenticated Users' or 'Domain Computers' groups:"
         foreach ($item in $GPOsMissingPermissions) {
             Write-Output "'$($item.DisplayName)'"
         }
-    }
-    Else {
-        Write-Output "There are no GPOs missing permissions for Authenticated Users AND Domain Computers."
+    } Else {
+        Write-Output 'There are no GPOs missing permissions for Authenticated Users AND Domain Computers.'
     }
 
     # Check for GPOs missing Authenticated Users
     $GPOsMissingAuthenticatedUsers = New-Object System.Collections.ArrayList
     foreach ($item in $GPOs) {
-        $GPOPermissionForAuthUsers = Get-GPPermission -Guid $item.Id -All | Select-Object -ExpandProperty Trustee | Where-Object {$_.Name -eq "Authenticated Users"}
+        $GPOPermissionForAuthUsers = Get-GPPermission -Guid $item.Id -All | Select-Object -ExpandProperty Trustee | Where-Object { $_.Name -eq 'Authenticated Users' }
         If (!$GPOPermissionForAuthUsers) {
-            $GPOsMissingAuthenticatedUsers.Add($item)| Out-Null
+            $GPOsMissingAuthenticatedUsers.Add($item) | Out-Null
         }
     }
     If ($GPOsMissingAuthenticatedUsers.Count -ne 0) {
-        Write-Warning  "The following Group Policy Objects do not grant any permissions to the 'Authenticated Users' security principal:"
+        Write-Warning "The following Group Policy Objects do not grant any permissions to the 'Authenticated Users' security principal:"
         foreach ($item in $GPOsMissingAuthenticatedUsers) {
             Write-Output "'$($item.DisplayName)'"
         }
-    }
-    Else {
-        Write-Output "There are no GPOs missing permissions for Authenticated Users."
+    } Else {
+        Write-Output 'There are no GPOs missing permissions for Authenticated Users.'
     }
 }
 

Active Directory/Get-OUDetails.ps1 → Active Directory/Get-OUDetail.ps1

@@ -1,12 +1,12 @@
-function Get-OUDetails {
+function Get-OUDetail {
     <#
     .SYNOPSIS
         Get advanced details about an organizational unit (OU) in Active Directory.
     .DESCRIPTION
         THIS IS STILL A CONCEPT WORK IN PROGRESS
 
     .NOTES
-        The Test-BlockInheritence, Test-IsCriticalSystemObject, and Test-IsHiddenOU functions were all created because
+        The Test-BlockInheritance, Test-IsCriticalSystemObject, and Test-IsHiddenOU functions were all created because
         I would rather display an explicit value (eg: $false) than a null that implies $false. Likewise, I prefer to
         display a $true or $false rather than 1 or a 0.
     #>
@@ -26,7 +26,7 @@ function Get-OUDetails {
             CanonicalName          = $OU.CanonicalName
             Parent                 = Get-ParentOU $OU
             Child                  = Get-ChildOU $OU
-            BlockInheritance       = Test-BlockInheritence $OU
+            BlockInheritance       = Test-BlockInheritance $OU
             CriticalLocation       = Test-IsCriticalSystemObject $OU
             ShowInAdvancedViewOnly = Test-IsHiddenOU $OU
         }
@@ -71,8 +71,8 @@ function Get-ChildOU {
     $ChildOU
 }
 
-function Test-BlockInheritence {
-    # Check if Block Inheritence is set on an organizational unit in Active Directory
+function Test-BlockInheritance {
+    # Check if Block Inheritance is set on an organizational unit in Active Directory
     [CmdletBinding()]
     [OutputType([Boolean])]
     param (

Active Directory/Get-OverlappingOUNames.ps1 → Active Directory/Get-OverlappingOUName.ps1

@@ -1,4 +1,4 @@
-function Get-OverlappingOUNames {
+function Get-OverlappingOUName {
     [CmdletBinding()]
     param (
 

DDI/Measure-DnsServerResponse.ps1

@@ -68,6 +68,7 @@ function Measure-NetworkHops {
     The server name or IP address to measure network hops to.
     #>
     [CmdletBinding()]
+    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '')]
     [OutputType([int])]
     param (
         [Parameter(