Merge pull request #757 from PowerShell/kapilmb/fix-align-assignment-rule

Kapil Borle · web-flow · commit 950e5f202e98 · 2017-05-09T14:16:34.000-07:00
Make AlignAssignmentStatement rule available on powershell core
diff --git a/Rules/AlignAssignmentStatement.cs b/Rules/AlignAssignmentStatement.cs
@@ -26,7 +26,7 @@ namespace Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules
 #if !CORECLR
     [Export(typeof(IScriptRule))]
 #endif
-    class AlignAssignmentStatement : ConfigurableRule
+    public class AlignAssignmentStatement : ConfigurableRule
     {
         // We keep this switch even though the rule has only one switch (this) as of now, because we want
         // to let the rule be expandable in the future to allow formatting assignments even
diff --git a/Utils/RuleMaker.psm1 b/Utils/RuleMaker.psm1
@@ -87,7 +87,7 @@ namespace Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules
     #if !CORECLR
     [Export(typeof(IScriptRule))]
 #endif
-    class {0} : IScriptRule
+    public class {0} : IScriptRule
     {{
         /// <summary>
         /// Analyzes the given ast to find the [violation]
@@ -483,4 +483,4 @@ Function Remove-Rule
 }
 
 Export-ModuleMember -Function Add-Rule
-Export-ModuleMember -Function Remove-Rule
+Export-ModuleMember -Function Remove-Rule

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ namespace Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules`
`26`	`26`	`#if !CORECLR`
`27`	`27`	`[Export(typeof(IScriptRule))]`
`28`	`28`	`#endif`
`29`		`- class AlignAssignmentStatement : ConfigurableRule`
	`29`	`+ public class AlignAssignmentStatement : ConfigurableRule`
`30`	`30`	`{`
`31`	`31`	`// We keep this switch even though the rule has only one switch (this) as of now, because we want`
`32`	`32`	`// to let the rule be expandable in the future to allow formatting assignments even`