-
-
Notifications
You must be signed in to change notification settings - Fork 207
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
add sentry to global usings #3043
Conversation
Instructions and example for changelogPlease add an entry to Example: ## Unreleased
- add sentry to global usings ([#3043](https://github.com/getsentry/sentry-dotnet/pull/3043)) If none of the above apply, you can opt out of this check by adding |
Actually the opt-out could be simpler, as Simon said here: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3043 +/- ##
=======================================
Coverage 76.39% 76.39%
=======================================
Files 351 351
Lines 13257 13257
Branches 2645 2645
=======================================
Hits 10128 10128
Misses 2450 2450
Partials 679 679 ☔ View full report in Codecov by Sentry. |
408151c
to
31073b1
Compare
Added back |
This clashes all over the place for me - and let alone our own types, obvious core library types - e.g.:
It is a huge assumption to think that library users want |
Now that this is complete:
ImplicitUsings
by default #1491We can enable namespace
Sentry
by default