commit prior merge to master

CONTRIBUTING.md

@@ -203,7 +203,7 @@ These templates are fully aligned to rule design, code and formatting style of t
 
 ## Repository folder structure
 
-See [DirectoryStructure.md](https://github.com/metablaster/WindowsFirewallRuleset/blob/master/Readme/DirectoryStructure.md)
+See [DirectoryStructure.md](https://github.com/metablaster/WindowsFirewallRuleset/blob/develop/Readme/DirectoryStructure.md)
 
 ## Final notes and where to start
 

Config/PSScriptAnalyzerSettings.psd1

@@ -138,8 +138,8 @@
 			# Checks if there is a space between a keyword and its corresponding open brace.
 			# E.g. foo { } instead of foo{ }
 			# default = true (powershell.codeFormatting.whitespaceBeforeOpenBrace)
-			# TODO: doesn't work as expected https://github.com/PowerShell/PSScriptAnalyzer/issues/1522
-			CheckOpenBrace = $false
+			# NOTE: works as expected https://github.com/PowerShell/PSScriptAnalyzer/issues/1522
+			CheckOpenBrace = $true
 			# Checks if there is space between a keyword and its corresponding open parenthesis.
 			#  E.g. if (true) instead of if(true)
 			# default = true (powershell.codeFormatting.whitespaceBeforeOpenParen)
@@ -164,8 +164,9 @@
 			# Checks if there is more than one space between parameters and values.
 			# E.g. foo -bar $baz -bat instead of foo  -bar $baz  -bat
 			# default = false (NOTE: Default value is $false at the moment due to the setting being new)
-			# TODO: this was disabled from true to false because of too many false positives
-			CheckParameter = $false
+			# NOTE: this was disabled from true to false because of too many false positives,
+			# it looks like it's working now
+			CheckParameter = $true
 		}
 
 		PSUseConsistentIndentation = @{