Add experimental diagnostics ids

Author: Martin Taillefer

Directory.Build.targets

@@ -35,9 +35,8 @@
     <!-- Prevent analyzer crashes from stopping things -->
     <NoWarn>$(NoWarn);AD0001</NoWarn>
 
-    <!-- R9A029 is for customers, not for this repo -->
-    <NoWarn>$(NoWarn);R9A029</NoWarn>
-
+    <!-- Experimental warnings are for customers, not for this repo -->
+    <NoWarn>$(NoWarn);EXTEXP0001;EXTEXP0002;EXTEXP0003;EXTEXP0004;EXTEXP0005;EXTEXP0006;EXTEXP0007;EXTEXP0008;EXTEXP0009;EXTEXP0010;EXTEXP0011</NoWarn>
 
     <!-- NU5104: A stable release of a package should not have a prerelease dependency -->
     <NoWarn>$(NoWarn);NU5104</NoWarn>

eng/MSBuild/Shared.props

@@ -1,6 +1,6 @@
 <Project>
   <ItemGroup>
-    <Compile Include="$(MSBuildThisFileDirectory)\..\..\src\Shared\WarningDefinitions.cs" LinkBase="Shared" />
+    <Compile Include="$(MSBuildThisFileDirectory)\..\..\src\Shared\DiagnosticIds\*.cs" LinkBase="Shared\DiagnosticIds" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(InjectSharedThrow)' == 'true'">

src/Analyzers/Microsoft.Extensions.ExtraAnalyzers/Common/CallAnalysis/CallAnalyzer.cs

@@ -17,7 +17,6 @@ public sealed partial class CallAnalyzer : DiagnosticAnalyzer
         DiagDescriptors.StartsEndsWith,
         DiagDescriptors.LegacyLogging,
         DiagDescriptors.StaticTime,
-        DiagDescriptors.StringFormat,
         DiagDescriptors.EnumStrings,
         DiagDescriptors.ValueTuple,
         DiagDescriptors.Arrays,
@@ -41,7 +40,6 @@ public override void Initialize(AnalysisContext context)
             _ = new StartsEndsWith(reg);
             _ = new StaticTime(reg);
             _ = new ValueTuple(reg);
-            _ = new StringFormat(reg);
             _ = new LegacyCollection(reg);
             _ = new Split(reg);
 

src/Analyzers/Microsoft.Extensions.ExtraAnalyzers/Common/CallAnalysis/StringFormat.cs

@@ -49,37 +49,7 @@ internal static class DiagDescriptors
         helpLinkUri: "https://TODO/r9a000",
         isEnabledByDefault: true);
 
-    // R9A001..R9A017 are retired
-
-    public static DiagnosticDescriptor StringFormat { get; } = new(
-        id: "R9A018",
-        messageFormat: Resources.StringFormatMessage,
-        title: Resources.StringFormatTitle,
-        category: Performance,
-        description: Resources.StringFormatDescription,
-        defaultSeverity: DiagnosticSeverity.Warning,
-        helpLinkUri: "https://TODO/r9a018",
-        isEnabledByDefault: true);
-
-    public static DiagnosticDescriptor UsingExcessiveDictionaryLookup { get; } = new(
-        id: "R9A019",
-        messageFormat: Resources.UsingExcessiveDictionaryLookupMessage,
-        title: Resources.UsingExcessiveDictionaryLookupTitle,
-        category: Performance,
-        description: Resources.UsingExcessiveDictionaryLookupDescription,
-        defaultSeverity: DiagnosticSeverity.Warning,
-        helpLinkUri: "https://TODO/r9a019",
-        isEnabledByDefault: true);
-
-    public static DiagnosticDescriptor UsingExcessiveSetLookup { get; } = new(
-        id: "R9A020",
-        messageFormat: Resources.UsingExcessiveSetLookupMessage,
-        title: Resources.UsingExcessiveSetLookupTitle,
-        category: Performance,
-        description: Resources.UsingExcessiveSetLookupDescription,
-        defaultSeverity: DiagnosticSeverity.Warning,
-        helpLinkUri: "https://TODO/r9a020",
-        isEnabledByDefault: true);
+    // R9A001..R9A020 are retired
 
     public static DiagnosticDescriptor UsingToStringInLoggers { get; } = new(
         id: "R9A021",
@@ -101,17 +71,7 @@ internal static class DiagDescriptors
         helpLinkUri: "https://TODO/r9a022",
         isEnabledByDefault: true);
 
-    // R9A023..R9A028 retired
-
-    public static DiagnosticDescriptor UsingExperimentalApi { get; } = new(
-        id: "R9A029",
-        messageFormat: Resources.UsingExperimentalApiMessage,
-        title: Resources.UsingExperimentalApiTitle,
-        category: Reliability,
-        description: Resources.UsingExperimentalApiDescription,
-        defaultSeverity: DiagnosticSeverity.Warning,
-        helpLinkUri: "https://TODO/r9a029",
-        isEnabledByDefault: true);
+    // R9A023..R9A029 retired
 
     public static DiagnosticDescriptor StartsEndsWith { get; } = new(
         id: "R9A030",
@@ -189,17 +149,7 @@ internal static class DiagDescriptors
         helpLinkUri: "https://TODO/r9a043",
         isEnabledByDefault: true);
 
-    public static DiagnosticDescriptor MakeArrayStatic { get; } = new(
-        id: "R9A044",
-        messageFormat: Resources.MakeArrayStaticMessage,
-        title: Resources.MakeArrayStaticTitle,
-        category: Performance,
-        description: Resources.MakeArrayStaticDescription,
-        defaultSeverity: DiagnosticSeverity.Warning,
-        helpLinkUri: "https://TODO/r9a044",
-        isEnabledByDefault: true);
-
-    // R9A045..R9A055 retired
+    // R9A044..R9A055 retired
 
     public static DiagnosticDescriptor AsyncCallInsideUsingBlock { get; } = new(
         id: "R9A056",