Convert public types to internal

Fixes #1502

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test/PublicApiTests.cs

@@ -0,0 +1,35 @@
+// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
+// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.
+
+namespace StyleCop.Analyzers.Test
+{
+    using System;
+    using System.Text;
+    using Xunit;
+
+    /// <summary>
+    /// Unit tests related to the public API surface of StyleCop.Analyzers.dll.
+    /// </summary>
+    public class PublicApiTests
+    {
+        /// <summary>
+        /// This test ensures all types in StyleCop.Analyzers.dll are marked internal.
+        /// </summary>
+        [Fact]
+        public void TestAllTypesAreInternal()
+        {
+            StringBuilder publicTypes = new StringBuilder();
+            foreach (Type type in typeof(AnalyzerCategory).Assembly.ExportedTypes)
+            {
+                if (publicTypes.Length > 0)
+                {
+                    publicTypes.Append(", ");
+                }
+
+                publicTypes.Append(type.Name);
+            }
+
+            Assert.Equal(string.Empty, publicTypes.ToString());
+        }
+    }
+}

StyleCop.Analyzers/StyleCop.Analyzers.Test/StyleCop.Analyzers.Test.csproj

@@ -274,6 +274,7 @@
     <Compile Include="OrderingRules\SA1217UnitTests.cs" />
     <Compile Include="OrderingRules\UsingCodeFixProviderUnitTests.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
+    <Compile Include="PublicApiTests.cs" />
     <Compile Include="ReadabilityRules\SA1100UnitTests.cs" />
     <Compile Include="ReadabilityRules\SA1101UnitTests.cs" />
     <Compile Include="ReadabilityRules\SA1102UnitTests.cs" />

StyleCop.Analyzers/StyleCop.Analyzers/AnalyzerExtensions.cs

@@ -13,7 +13,7 @@ namespace StyleCop.Analyzers
     /// <summary>
     /// Provides extension methods to deal for analyzers.
     /// </summary>
-    public static class AnalyzerExtensions
+    internal static class AnalyzerExtensions
     {
         /// <summary>
         /// A cache of the result of computing whether a document has an auto-generated header.

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/ElementDocumentationSummaryBase.cs

@@ -13,7 +13,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// <summary>
     /// This is the base class for analyzers which examine the <c>&lt;summary&gt;</c> text of a documentation comment.
     /// </summary>
-    public abstract class ElementDocumentationSummaryBase : DiagnosticAnalyzer
+    internal abstract class ElementDocumentationSummaryBase : DiagnosticAnalyzer
     {
         /// <inheritdoc/>
         public override void Initialize(AnalysisContext context)

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/FileHeaderAnalyzers.cs

@@ -26,7 +26,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// <seealso href="https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1641.md">SA1641 File header company name text must match</seealso>
     /// <seealso href="https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1649.md">SA1649 File header file name documentation must match type name</seealso>
     [DiagnosticAnalyzer(LanguageNames.CSharp)]
-    public class FileHeaderAnalyzers : DiagnosticAnalyzer
+    internal class FileHeaderAnalyzers : DiagnosticAnalyzer
     {
         private const string SA1633Identifier = "SA1633";
         private const string SA1634Identifier = "SA1634";

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/FileHeaderCodeFixProvider.cs

@@ -24,7 +24,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// </remarks>
     [ExportCodeFixProvider(LanguageNames.CSharp, Name = nameof(FileHeaderCodeFixProvider))]
     [Shared]
-    public class FileHeaderCodeFixProvider : CodeFixProvider
+    internal class FileHeaderCodeFixProvider : CodeFixProvider
     {
         /// <inheritdoc/>
         public override ImmutableArray<string> FixableDiagnosticIds { get; }

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/InheritdocCodeFixProvider.cs

@@ -22,7 +22,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// </summary>
     [ExportCodeFixProvider(LanguageNames.CSharp, Name = nameof(InheritdocCodeFixProvider))]
     [Shared]
-    public class InheritdocCodeFixProvider : CodeFixProvider
+    internal class InheritdocCodeFixProvider : CodeFixProvider
     {
         /// <inheritdoc/>
         public override ImmutableArray<string> FixableDiagnosticIds { get; } =

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/PartialElementDocumentationSummaryBase.cs

@@ -14,7 +14,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// This is the base class for analyzers which examine the <c>&lt;summary&gt;</c> or <c>&lt;content&gt;</c> text of
     /// the documentation comment associated with a <c>partial</c> element.
     /// </summary>
-    public abstract class PartialElementDocumentationSummaryBase : DiagnosticAnalyzer
+    internal abstract class PartialElementDocumentationSummaryBase : DiagnosticAnalyzer
     {
         /// <inheritdoc/>
         public override void Initialize(AnalysisContext context)

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/PropertyDocumentationSummaryBase.cs

@@ -12,7 +12,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// <summary>
     /// This is the base class for analyzers which examine the <c>&lt;value&gt;</c> text of a documentation comment on a property declaration.
     /// </summary>
-    public abstract class PropertyDocumentationSummaryBase : DiagnosticAnalyzer
+    internal abstract class PropertyDocumentationSummaryBase : DiagnosticAnalyzer
     {
         /// <inheritdoc/>
         public override void Initialize(AnalysisContext context)

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/SA1600ElementsMustBeDocumented.cs

@@ -26,7 +26,7 @@ namespace StyleCop.Analyzers.DocumentationRules
     /// delegates, enums, events, finalizers, indexers, interfaces, methods, properties, and structs.</para>
     /// </remarks>
     [DiagnosticAnalyzer(LanguageNames.CSharp)]
-    public class SA1600ElementsMustBeDocumented : DiagnosticAnalyzer
+    internal class SA1600ElementsMustBeDocumented : DiagnosticAnalyzer
     {
         /// <summary>
         /// The ID for diagnostics produced by the <see cref="SA1600ElementsMustBeDocumented"/> analyzer.