getsentry / sentry-dart

Sentry SDK for Dart and Flutter
https://sentry.io/for/flutter/
MIT License
725 stars 223 forks source link

Migrates to package:web and js_interop #2104

Closed denrase closed 1 week ago

denrase commented 1 week ago

:scroll: Description

Continuation of https://github.com/getsentry/sentry-dart/pull/2064 by josh-burton

Additions:

Updated & Re-run ffigen to generate bindings with updated parameters.

:bulb: Motivation and Context

Supports compilation via WASM

:green_heart: How did you test it?

Make sure CI runs without issues.

:pencil: Checklist

:crystal_ball: Next steps

This is a rather large bump for us, and we will probably create a pre-release version if we choose to go with this approach.

Still to resolve

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 88.63%. Comparing base (3cac5bf) to head (7bb2685).

Files Patch % Lines
...tter/lib/src/native/cocoa/sentry_native_cocoa.dart 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## v9 #2104 +/- ## ========================================== - Coverage 95.25% 88.63% -6.62% ========================================== Files 54 220 +166 Lines 1791 7623 +5832 ========================================== + Hits 1706 6757 +5051 - Misses 85 866 +781 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

buenaflor commented 1 week ago

We can close this now as we don't plan to bump the versions