-
-
Notifications
You must be signed in to change notification settings - Fork 230
Remove Obsolete Extra members #4640
Copy link
Copy link
Open
Labels
.NETPull requests that update .net codePull requests that update .net codeTaskvNextBinary/Source/Behavioral Breaking Changes scheduled for the next Major release.Binary/Source/Behavioral Breaking Changes scheduled for the next Major release.
Metadata
Metadata
Assignees
Labels
.NETPull requests that update .net codePull requests that update .net codeTaskvNextBinary/Source/Behavioral Breaking Changes scheduled for the next Major release.Binary/Source/Behavioral Breaking Changes scheduled for the next Major release.
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
No status
Description
SentrySpan,SentryTransactionandTransactionTracerall haveExtramembers like:sentry-dotnet/src/Sentry/SentrySpan.cs
Lines 82 to 84 in 748fcfe
sentry-dotnet/src/Sentry/SentrySpan.cs
Lines 86 to 88 in 748fcfe
Although we do probably want to get rid of these, as believe we also want to get rid of the
Datamembers 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