Merge pull request #3000 from sharwell/rm-suppression

Use code instead of pragma to document unused parameter
DotNetAnalyzers · Sep 3, 2019 · 1b968a5 · 1b968a5
2 parents d846215 + bbd5142
commit 1b968a5
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers/Helpers/AccessLevelHelper.cs b/StyleCop.Analyzers/StyleCop.Analyzers/Helpers/AccessLevelHelper.cs
@@ -292,12 +292,13 @@ internal static Accessibility GetDeclaredAccessibility(this BaseFieldDeclaration
             return declaredSymbol?.DeclaredAccessibility ?? Accessibility.NotApplicable;
         }
 
-#pragma warning disable IDE0060 // Remove unused parameter - false positive
         internal static Accessibility GetDeclaredAccessibility(this EnumMemberDeclarationSyntax syntax)
         {
+            // Intentionally unused
+            _ = syntax;
+
             return Accessibility.Public;
         }
-#pragma warning restore IDE0060 // Remove unused parameter
 
         internal static Accessibility GetDeclaredAccessibility(this DelegateDeclarationSyntax syntax, SemanticModel semanticModel, CancellationToken cancellationToken)
         {