Ensure that analyzers in CodeStyle layer sub-type AbstractBuiltInCodeStyleDiagnosticAnalyzer or AbstractCodeQualityDiagnosticAnalyzer, not DiagnosticAnalyzer

[mavasani]

@Youssef1313 One example:

roslyn/src/Analyzers/Core/Analyzers/RemoveUnnecessaryImports/AbstractRemoveUnnecessaryImportsDiagnosticAnalyzer.cs

Lines 26 to 27 in 64769f8

	internal abstract class AbstractRemoveUnnecessaryImportsDiagnosticAnalyzer
	: DiagnosticAnalyzer, IBuiltInAnalyzer

This is turn causes IDE0005 diagnostics to not have an associated help link, falling back to bing search instead of https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0005:

Let me file an issue and tag you, in case you are interested in fixing this up

Originally posted by @mavasani in #49995 (comment)

[mavasani]

@Youssef1313

[mavasani]

One way to enforce this could be to ban usage of DiagnosticDescriptor constructor in the CodeStyle folder by adding its doc comment ID to https://github.com/dotnet/roslyn/blob/master/src/CodeStyle/BannedSymbols.txt. We can add a suppression for this banned symbol usage in AbstractBuiltInCodeStyleDiagnosticAnalyzer and AbstractCodeQualityDiagnosticAnalyzer types, but get an analyzer diagnostic if any other code directly inherits from DiagnosticAnalyzer type and instantiates DiagnosticDescriptor.