[Proposal] [.NET 8] Use `ExperimentalAttribute` instead of `ObsoleteAttribute` to indicate alpha APIs. about dotnet-sdk

[halspang]

@philliphoff - Would this be a breaking change if we switched them all over? Or is there a way to avoid that?

Or, is this proposal just to use it for new features and the old features will be [Obsolete] until they're no longer preview?

[philliphoff]

@halspang Yes, it would be breaking in that it's a different compiler/analyzer error to disable. That said, I just discovered that .NET 8 includes a new ExperimentalAttribute which seems even closer to what we want. (RequiresPreviewFeaturesAttribute was originally targeted at APIs in .NET itself but would still have been better than ObsoleteAttribute.) I suggest we wait until .NET 6 support is dropped and then switch to ExperimentalAttribute, as a whole, so users needn't have to change how they suppress the warnings more than once.

[Neme12]

@philliphoff Couldn't the analyzer just suggest using ExperimentalAttribute if it exists? That means for users on .NET 8 and others who define it. From what I understand, others can define it in their own code as well even on a lower TFM, just like many other attributes that the compiler understands - they're not required to come from mscorlib.

Also, regardless of whether it supports the atribute being defined manually (which it should IMO, there's no reason not to - it's simple to just check whether a type named System.Diagnostics.CodeAnalysis.ExperimentalAttribute exists), why wait until .NET 6 support is dropped when .NET 8 already exists and the analyzer could already work on .NET 8?

[Neme12]

Oh, this isn't about an analyzer, my bad.