Skip to content

Support Url and Message parameters in RequiresPreviewFeaturesAttribute #5502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
buyaa-n merged 26 commits into from
Oct 7, 2021
Merged

Support Url and Message parameters in RequiresPreviewFeaturesAttribute #5502

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
947927a
First cut of message and URL support. Needs to be cleaned up
Sep 16, 2021
0dac989
GeneralPreviewFeatureAttributeRule and unit test changes
Sep 17, 2021
1d70485
ImplementsPreviewInterfaceRule and unit test changes
Sep 17, 2021
e4105f9
UsesPreviewTypeParameterRule and unit test changes
Sep 17, 2021
4232a34
Everything else
Sep 17, 2021
3ec9901
Clean up
Sep 17, 2021
1732835
Fix test failures in PCA
buyaa-n Sep 20, 2021
06cd8cc
Self review + unit tests
Sep 20, 2021
1bc3edf
Make method static
Sep 21, 2021
712bfbc
First cut of addressing comments
Sep 23, 2021
5fefe10
Before reordering args
Sep 23, 2021
722716e
Checkpoint
Sep 23, 2021
eb77b71
All unit tests pass
Sep 24, 2021
f94f4b9
MInor cleanup
Sep 24, 2021
7e578a0
More cleanup
Sep 24, 2021
86bb96c
Before unit test regex changes
Sep 24, 2021
4eec26c
Bug fix and all unit tests pass
Sep 24, 2021
abbf59f
sq
Sep 24, 2021
969faa1
sq
Sep 24, 2021
7df6264
Address comments
Sep 28, 2021
98ee43d
sq
Sep 28, 2021
91aae98
pack
Sep 28, 2021
70151db
Merge conflicts
buyaa-n Oct 6, 2021
96db9b0
Omit quotes around urls in diagnostic messages
jeffhandley Oct 7, 2021
cd84f30
Formatting tweaks
jeffhandley Oct 7, 2021
4339b4a
Merge conflicts
buyaa-n Oct 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
MInor cleanup
  • Loading branch information
Prashanth Govindarajan committed Sep 24, 2021
commit f94f4b9a329e1cea176defc8b00f458fbea1e8f8
9 changes: 0 additions & 9 deletions src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/Runtime/DetectPreviewFeatureAnalyzer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,6 @@ public abstract class DetectPreviewFeatureAnalyzer : DiagnosticAnalyzer
internal const string DefaultURL = "https://aka.ms/dotnet-warnings/preview-features";
private static readonly LocalizableString s_localizableTitle = CreateLocalizableResourceString(nameof(DetectPreviewFeaturesTitle));
private static readonly LocalizableString s_localizableDescription = CreateLocalizableResourceString(nameof(DetectPreviewFeaturesDescription));
//internal static readonly LocalizableString s_detectPreviewFeaturesMessage = CreateLocalizableResourceString(nameof(DetectPreviewFeaturesMessage));
//internal static readonly LocalizableString s_implementsPreviewInterfaceMessage = CreateLocalizableResourceString(nameof(ImplementsPreviewInterfaceMessage));
//internal static readonly LocalizableString s_implementsPreviewMethodMessage = CreateLocalizableResourceString(nameof(ImplementsPreviewMethodMessage));
//internal static readonly LocalizableString s_overridePreviewMethodMessage = CreateLocalizableResourceString(nameof(OverridesPreviewMethodMessage));
//internal static readonly LocalizableString s_derivesFromPreviewClassMessage = CreateLocalizableResourceString(nameof(DerivesFromPreviewClassMessage));
//internal static readonly LocalizableString s_usesPreviewTypeParameterMessage = CreateLocalizableResourceString(nameof(UsesPreviewTypeParameterMessage));
//internal static readonly LocalizableString s_methodReturnsPreviewTypeMessage = CreateLocalizableResourceString(nameof(MethodReturnsPreviewTypeMessage));
//internal static readonly LocalizableString s_methodUsesPreviewTypeAsParameterMessage = CreateLocalizableResourceString(nameof(MethodUsesPreviewTypeAsParamaterMessage));
//internal static readonly LocalizableString s_fieldOrEventIsPreviewTypeMessage = CreateLocalizableResourceString(nameof(FieldIsPreviewTypeMessage));
private static readonly ImmutableArray<SymbolKind> s_symbols = ImmutableArray.Create(SymbolKind.NamedType, SymbolKind.Method, SymbolKind.Property, SymbolKind.Field, SymbolKind.Event);

internal static DiagnosticDescriptor GeneralPreviewFeatureAttributeRule = DiagnosticDescriptorHelper.Create(RuleId,
Expand Down