Skip to content

Commit

Permalink
Merge pull request #4355 from mavasani/CompilationEnd_CustomTag
Browse files Browse the repository at this point in the history
Add CompilationEnd custom tag to diagnostics reported in compilation end actions
  • Loading branch information
mavasani committed Oct 23, 2020
2 parents 2823fc9 + a4193e7 commit 9b1b3df
Show file tree
Hide file tree
Showing 79 changed files with 830 additions and 477 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
{
"tool": {
"name": "MetaCompilation.Analyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ public sealed class FixerWithFixAllAnalyzer : DiagnosticAnalyzer
DiagnosticSeverity.Warning,
description: s_localizableOverrideGetFixAllProviderDescription,
isEnabledByDefault: true,
customTags: WellKnownDiagnosticTags.Telemetry);
customTags: WellKnownDiagnosticTagsExtensions.CompilationEndAndTelemetry);

public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics =>
ImmutableArray.Create(CreateCodeActionEquivalenceKeyRule, OverrideCodeActionEquivalenceKeyRule, OverrideGetFixAllProviderRule);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public abstract class DiagnosticAnalyzerApiUsageAnalyzer<TTypeSyntax> : Diagnost
DiagnosticSeverity.Warning,
isEnabledByDefault: true,
description: s_localizableDescription,
customTags: WellKnownDiagnosticTags.Telemetry);
customTags: WellKnownDiagnosticTagsExtensions.CompilationEndAndTelemetry);

public static readonly DiagnosticDescriptor DoNotUseTypesFromAssemblyIndirectRule = new DiagnosticDescriptor(
DiagnosticIds.DoNotUseTypesFromAssemblyRuleId,
Expand All @@ -45,7 +45,7 @@ public abstract class DiagnosticAnalyzerApiUsageAnalyzer<TTypeSyntax> : Diagnost
DiagnosticSeverity.Warning,
isEnabledByDefault: true,
description: s_localizableDescription,
customTags: WellKnownDiagnosticTags.Telemetry);
customTags: WellKnownDiagnosticTagsExtensions.CompilationEndAndTelemetry);

protected abstract bool IsNamedTypeDeclarationBlock(SyntaxNode syntax);
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(DoNotUseTypesFromAssemblyDirectRule, DoNotUseTypesFromAssemblyIndirectRule);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ public sealed class DiagnosticDescriptorCreationAnalyzer : DiagnosticAnalyzer
DiagnosticSeverity.Warning,
isEnabledByDefault: true,
description: s_localizableUseUniqueDiagnosticIdDescription,
customTags: WellKnownDiagnosticTags.Telemetry);
customTags: WellKnownDiagnosticTagsExtensions.CompilationEndAndTelemetry);

public static readonly DiagnosticDescriptor UseCategoriesFromSpecifiedRangeRule = new DiagnosticDescriptor(
DiagnosticIds.UseCategoriesFromSpecifiedRangeRuleId,
Expand All @@ -122,7 +122,7 @@ public sealed class DiagnosticDescriptorCreationAnalyzer : DiagnosticAnalyzer
DiagnosticSeverity.Warning,
isEnabledByDefault: true,
description: s_localizableAnalyzerCategoryAndIdRangeFileInvalidDescription,
customTags: WellKnownDiagnosticTags.Telemetry);
customTags: WellKnownDiagnosticTagsExtensions.CompilationEndAndTelemetry);

public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(
UseLocalizableStringsInDescriptorRule,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
{
"tool": {
"name": "Microsoft.CodeAnalysis.Analyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand Down Expand Up @@ -146,6 +146,7 @@
"Visual Basic"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand Down Expand Up @@ -200,6 +201,7 @@
"Visual Basic"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand Down Expand Up @@ -236,6 +238,7 @@
"Visual Basic"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand Down Expand Up @@ -299,7 +302,7 @@
{
"tool": {
"name": "Microsoft.CodeAnalysis.CSharp.Analyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand Down Expand Up @@ -452,6 +455,7 @@
"C#"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand Down Expand Up @@ -479,7 +483,7 @@
{
"tool": {
"name": "Microsoft.CodeAnalysis.VisualBasic.Analyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand Down Expand Up @@ -615,6 +619,7 @@
"Visual Basic"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ internal static class SymbolIsBannedAnalyzer
isEnabledByDefault: true,
description: BannedApiAnalyzerResources.DuplicateBannedSymbolDescription,
helpLinkUri: "https://github.com/dotnet/roslyn-analyzers/blob/master/src/Microsoft.CodeAnalysis.BannedApiAnalyzers/BannedApiAnalyzers.Help.md",
customTags: WellKnownDiagnosticTags.Telemetry);
customTags: WellKnownDiagnosticTagsExtensions.CompilationEndAndTelemetry);
}

public abstract class SymbolIsBannedAnalyzer<TSyntaxKind> : DiagnosticAnalyzer
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
{
"tool": {
"name": "Microsoft.CodeAnalysis.BannedApiAnalyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand All @@ -14,7 +14,7 @@
{
"tool": {
"name": "Microsoft.CodeAnalysis.CSharp.BannedApiAnalyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand Down Expand Up @@ -50,6 +50,7 @@
"C#"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand All @@ -76,7 +77,7 @@
{
"tool": {
"name": "Microsoft.CodeAnalysis.VisualBasic.BannedApiAnalyzers",
"version": "2.9.9",
"version": "2.9.12",
"language": "en-US"
},
"rules": {
Expand Down Expand Up @@ -112,6 +113,7 @@
"Visual Basic"
],
"tags": [
"CompilationEnd",
"Telemetry"
]
}
Expand Down
Loading

0 comments on commit 9b1b3df

Please sign in to comment.