sentry-dotnet icon indicating copy to clipboard operation
sentry-dotnet copied to clipboard

experimental wpf integration

Open bruno-garcia opened this issue 3 years ago • 2 comments

Starting from a sample to see what hooks we need. The goal is to create a new package Sentry.Wpf that already hooks to DispatcherUnhandledException. Additionally it would set up offline caching by default, and create some transactions out of the box.

Once #1013 is done it can also emit release health stats out of the box.

Blocked by #1095

bruno-garcia avatar Jun 02 '21 03:06 bruno-garcia

Codecov Report

Merging #1036 (b93554f) into main (7604242) will decrease coverage by 0.20%. The diff coverage is n/a.

:exclamation: Current head b93554f differs from pull request most recent head a6f3725. Consider uploading reports for the commit a6f3725 to get more accurate results

@@            Coverage Diff             @@
##             main    #1036      +/-   ##
==========================================
- Coverage   84.34%   84.13%   -0.21%     
==========================================
  Files         219      219              
  Lines        7423     7350      -73     
  Branches     1417     1409       -8     
==========================================
- Hits         6261     6184      -77     
+ Misses        735      730       -5     
- Partials      427      436       +9     
Impacted Files Coverage Δ
...als/DiagnosticSource/SentryDiagnosticSubscriber.cs 60.00% <0.00%> (-35.46%) :arrow_down:
src/Sentry/Infrastructure/TraceDiagnosticLogger.cs 72.72% <0.00%> (-27.28%) :arrow_down:
src/Sentry/Internal/Extensions/StreamExtensions.cs 89.47% <0.00%> (-10.53%) :arrow_down:
src/Sentry/Reflection/AssemblyExtensions.cs 66.66% <0.00%> (-10.26%) :arrow_down:
....EntityFramework/SentryQueryPerformanceListener.cs 76.19% <0.00%> (-10.18%) :arrow_down:
src/Sentry/Internal/SdkComposer.cs 70.21% <0.00%> (-4.26%) :arrow_down:
src/Sentry/Internal/Http/RetryAfterHandler.cs 95.83% <0.00%> (-4.17%) :arrow_down:
...Internals/DiagnosticSource/SentryEFCoreListener.cs 89.28% <0.00%> (-3.74%) :arrow_down:
src/Sentry/Internal/MonoSentryStackTraceFactory.cs 92.59% <0.00%> (-3.71%) :arrow_down:
...rc/Sentry/Extensibility/SentryStackTraceFactory.cs 89.24% <0.00%> (-2.89%) :arrow_down:
... and 31 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Jan 05 '22 23:01 codecov-commenter

Fails
:no_entry_sign: Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

- experimental wpf integration ([#1036](https://github.com/getsentry/sentry-dotnet/pull/1036))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by :no_entry_sign: dangerJS against a6f3725340216d9c60203be8a62454d77efe0dfa

github-actions[bot] avatar Jan 05 '22 23:01 github-actions[bot]

@SimonCropp just going through the old pull requests, looking to close any that are no longer active. Is this one still being worked on or should we close this?

jamescrosswell avatar Jul 07 '23 06:07 jamescrosswell