-
-
Notifications
You must be signed in to change notification settings - Fork 229
Open
Labels
.NETPull requests that update .net codePull requests that update .net code
Description
Description
SentrySpan, SentryTransaction and TransactionTracer all have Extra members like:
sentry-dotnet/src/Sentry/SentrySpan.cs
Lines 82 to 84 in 748fcfe
| /// <inheritdoc /> | |
| [Obsolete("Use Data")] | |
| public IReadOnlyDictionary<string, object?> Extra => Data; |
sentry-dotnet/src/Sentry/SentrySpan.cs
Lines 86 to 88 in 748fcfe
| /// <inheritdoc /> | |
| [Obsolete("Use SetData")] | |
| public void SetExtra(string key, object? value) => SetData(key, value); |
Although we do probably want to get rid of these, as believe we also want to get rid of the Data members that we're recommending people switch to... since I think all of these will be replaced by Attributes at some stage right?
Possibly we delay replacing these until the new Attributes are available and, at that point, we could ship an Analyzer and a Code fix to help people migrate (since Extras may be used quite widely).
See Also
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
.NETPull requests that update .net codePull requests that update .net code
Projects
Status
No status