Skip to content

Commit e1ae1ef

Browse files
committed
Silence SA1515 because it is too dogmatic in practice
1 parent 9c5cc02 commit e1ae1ef

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

Tdg5.StandardConventions.Tests/Data/StyleCopRules/BlockScopedExpectations.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -568,7 +568,8 @@ public static void SA1512_SingleLineCommentsMustNotBeFollowedByBlankLine()
568568
/// A method that contains a single line comment that is not preceded by a
569569
/// blank line.
570570
/// </summary>
571-
[CodeAnalysisViolationExpected("SA1515", "Warning")]
571+
[CodeAnalysisViolationExpected(
572+
"SA1515", "Warning", disabledReason: "SA1515 is silenced")]
572573
public static void SA1515_SingleLineCommentMustBePrecededByBlankLine()
573574
{
574575
static int Method() => 0;

Tdg5.StandardConventions/Tdg5.StandardConventions.globalconfig

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -804,7 +804,8 @@ dotnet_diagnostic.SA1513.severity = none
804804
dotnet_diagnostic.SA1514.severity = warning
805805

806806
# SA1515: Single-line comment should be preceded by blank line.
807-
dotnet_diagnostic.SA1515.severity = warning
807+
# Motivation: In practice, this is too dogmatic.
808+
dotnet_diagnostic.SA1515.severity = silent
808809

809810
# SA1516: Elements should be separated by blank line.
810811
dotnet_diagnostic.SA1516.severity = warning

0 commit comments

Comments
 (0)