We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Set Up is quite complex:
In some platforms you need Sentry.Profiling in some you don't and others neither are supported We have series of notes and warnings, and tabs:
Sentry.Profiling
Ideally we have a compiler error if Sentry.Profiling is added to an unsupported version, so it's clear Sentry.Profiling isn't supported on Android
Customers are copying the snippet and adding both packages without picking on the fine print and having a bad experience during set up.
The text was updated successfully, but these errors were encountered:
Can we do that with a bit of MSBuild magic? Can we pick up the targeted platform there together with what packages are installed?
Sorry, something went wrong.
@jamescrosswell raised a good point: We could put the platform specific options behind guards like we do here
sentry-dotnet/src/Sentry/SentryOptions.cs
Lines 26 to 28 in a8859d8
this should do the trick 🤞 https://github.com/getsentry/sentry-dotnet/pull/3371/files
vaind
Successfully merging a pull request may close this issue.
Set Up is quite complex:
In some platforms you need
Sentry.Profiling
in some you don't and others neither are supportedWe have series of notes and warnings, and tabs:
Ideally we have a compiler error if
Sentry.Profiling
is added to an unsupported version, so it's clearSentry.Profiling
isn't supported on AndroidCustomers are copying the snippet and adding both packages without picking on the fine print and having a bad experience during set up.
The text was updated successfully, but these errors were encountered: