Remove redundant rule configuration

We already configure the rule when initializing ScriptAnalyzer. Hence, configuring them within their respective AnalyzeAst methods is redundant.

Author: Kapil Borle

Rules/PlaceCloseBrace.cs

@@ -44,11 +44,6 @@ public override IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string file
                 throw new ArgumentNullException("ast");
             }
 
-            if (!IsRuleConfigured)
-            {
-                ConfigureRule();
-            }
-
             if (!Enable)
             {
                 return Enumerable.Empty<DiagnosticRecord>();

Rules/PlaceOpenBrace.cs

@@ -51,13 +51,7 @@ public override IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string file
                 throw new ArgumentNullException("ast");
             }
 
-            if (!IsRuleConfigured)
-            {
-                ConfigureRule();
-            }
-
             var diagnosticRecords = new List<DiagnosticRecord>();
-
             if (Enable)
             {
                 foreach (var violationFinder in violationFinders)

Rules/UseConsistentIndentation.cs

@@ -49,11 +49,6 @@ public override IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string file
                 throw new ArgumentNullException("ast");
             }
 
-            if (!IsRuleConfigured)
-            {
-                ConfigureRule();
-            }
-
             // we add this switch because there is no clean way
             // to disable the rule by default
             if (!Enable)