Configure and fix SA1518 at end of file

sharwell · sharwell · commit ed741772c36f · 2016-01-01T17:06:54.000-06:00
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.CodeFixes/ReadabilityRules/SA1128CodeFixProvider.cs b/StyleCop.Analyzers/StyleCop.Analyzers.CodeFixes/ReadabilityRules/SA1128CodeFixProvider.cs
@@ -116,4 +116,4 @@ protected override async Task<SyntaxNode> FixAllInDocumentAsync(FixAllContext fi
             }
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/ExportCodeFixProviderAttributeNameTest.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/ExportCodeFixProviderAttributeNameTest.cs
@@ -47,4 +47,4 @@ public void TestExportCodeFixProviderAttribute(Type codeFixProvider)
             Assert.Equal(LanguageNames.CSharp, exportCodeFixProviderAttribute.Languages[0]);
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1516UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1516UnitTests.cs
@@ -553,4 +553,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
             return new SA1516CodeFixProvider();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1401UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1401UnitTests.cs
@@ -94,4 +94,4 @@ protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers(
             yield return new SA1401FieldsMustBePrivate();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1407UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1407UnitTests.cs
@@ -237,4 +237,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
             return new SA1407SA1408CodeFixProvider();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1408UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1408UnitTests.cs
@@ -221,4 +221,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
             return new SA1407SA1408CodeFixProvider();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1410UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1410UnitTests.cs
@@ -109,4 +109,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
             return new SA1410SA1411CodeFixProvider();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1411UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1411UnitTests.cs
@@ -165,4 +165,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
             return new SA1410SA1411CodeFixProvider();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/NamingRules/SA1302UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/NamingRules/SA1302UnitTests.cs
@@ -349,4 +349,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
             return new SA1302CodeFixProvider();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/ReadabilityRules/SA1115UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/ReadabilityRules/SA1115UnitTests.cs
@@ -1249,4 +1249,4 @@ protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers(
             yield return new SA1115ParameterMustFollowComma();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/ReadabilityRules/SA1118UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/ReadabilityRules/SA1118UnitTests.cs
@@ -320,4 +320,4 @@ protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers(
             yield return new SA1118ParameterMustNotSpanMultipleLines();
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/SA1604ElementDocumentationMustHaveSummary.cs b/StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/SA1604ElementDocumentationMustHaveSummary.cs
@@ -79,4 +79,4 @@ protected override void HandleXmlElement(SyntaxNodeAnalysisContext context, Docu
             }
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers/MaintainabilityRules/SA1405DebugAssertMustProvideMessageText.cs b/StyleCop.Analyzers/StyleCop.Analyzers/MaintainabilityRules/SA1405DebugAssertMustProvideMessageText.cs
@@ -60,4 +60,4 @@ private static void HandleInvocationExpression(SyntaxNodeAnalysisContext context
             HandleMethodCall(context, nameof(Debug.Assert), 1, Descriptor);
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers/MaintainabilityRules/SA1407ArithmeticExpressionsMustDeclarePrecedence.cs b/StyleCop.Analyzers/StyleCop.Analyzers/MaintainabilityRules/SA1407ArithmeticExpressionsMustDeclarePrecedence.cs
@@ -122,4 +122,4 @@ private static bool IsSameFamily(SyntaxToken operatorToken1, SyntaxToken operato
             return isSameFamily;
         }
     }
-}
+}
diff --git a/StyleCop.Analyzers/stylecop.json b/StyleCop.Analyzers/stylecop.json
@@ -6,6 +6,9 @@
       "copyrightText": "Copyright (c) {companyName}. All Rights Reserved.\r\nLicensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.",
       "xmlHeader": false,
       "fileNamingConvention": "metadata"
+    },
+    "layoutRules": {
+      "newlineAtEndOfFile": "require"
     }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -116,4 +116,4 @@ protected override async Task<SyntaxNode> FixAllInDocumentAsync(FixAllContext fi`
`116`	`116`	`}`
`117`	`117`	`}`
`118`	`118`	`}`
`119`		`-}`
	`119`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -47,4 +47,4 @@ public void TestExportCodeFixProviderAttribute(Type codeFixProvider)`
`47`	`47`	`Assert.Equal(LanguageNames.CSharp, exportCodeFixProviderAttribute.Languages[0]);`
`48`	`48`	`}`
`49`	`49`	`}`
`50`		`-}`
	`50`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -553,4 +553,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()`
`553`	`553`	`return new SA1516CodeFixProvider();`
`554`	`554`	`}`
`555`	`555`	`}`
`556`		`-}`
	`556`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -94,4 +94,4 @@ protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers(`
`94`	`94`	`yield return new SA1401FieldsMustBePrivate();`
`95`	`95`	`}`
`96`	`96`	`}`
`97`		`-}`
	`97`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -237,4 +237,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()`
`237`	`237`	`return new SA1407SA1408CodeFixProvider();`
`238`	`238`	`}`
`239`	`239`	`}`
`240`		`-}`
	`240`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -221,4 +221,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()`
`221`	`221`	`return new SA1407SA1408CodeFixProvider();`
`222`	`222`	`}`
`223`	`223`	`}`
`224`		`-}`
	`224`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -109,4 +109,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()`
`109`	`109`	`return new SA1410SA1411CodeFixProvider();`
`110`	`110`	`}`
`111`	`111`	`}`
`112`		`-}`
	`112`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -165,4 +165,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()`
`165`	`165`	`return new SA1410SA1411CodeFixProvider();`
`166`	`166`	`}`
`167`	`167`	`}`
`168`		`-}`
	`168`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -349,4 +349,4 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()`
`349`	`349`	`return new SA1302CodeFixProvider();`
`350`	`350`	`}`
`351`	`351`	`}`
`352`		`-}`
	`352`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -1249,4 +1249,4 @@ protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers(`
`1249`	`1249`	`yield return new SA1115ParameterMustFollowComma();`
`1250`	`1250`	`}`
`1251`	`1251`	`}`
`1252`		`-}`
	`1252`	`+}`