-
-
Notifications
You must be signed in to change notification settings - Fork 210
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.
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
Rename types likely to clash to include ImplicitUsings
by default
#1491
Comments
Clear example, Before simply renaming everything, we should re-think our type names within the context of implict/global usings. Does the whole .NET ecosystem need to re-think this? Are these conversations already happening, and can we align to what others might be doing? |
Discussed. I think we should do this for the next major (4.0), especially in light of global usings. Another good example is Lets do this as part of reviewing the public API surface for 4.0. |
See also #1585 |
We should give this one a try now on 4.0.0. Adding Wdyt @bitsandfoxes @jamescrosswell ? |
From memory, we initially pushed this back to 5.0.0 from 4.0.0 to descope the 4.0 release a bit and make sure we'd be able to ship it around the same time as the .net8.0 release. I think we've derisked that now so could potentially put this one back in there. It'd be an easy first issue to pick up. |
Since the 3.x SDK works fine on .NET 8, releasing 4.0.0 is not time sensitive. So if we want to extend the scope/time to take advantage of the major bump, we can |
What about |
Since that'll be more prevalent going forward, to keep it in sync with most SDKs, I think it's fine we keep it as-is. |
See: #1487
Candidates
Sentry.User
toSentryUser
#3015Sentry.Runtime
toSentryRuntime
#3016Sentry.Span
toSentry.SentrySpan
#3021Sentry.Transaction
toSentry.SentryTransaction
#3023The text was updated successfully, but these errors were encountered: