Skip to content

ref(profiling): support symbolication of native frames in react-native#58771

Merged
viglia merged 14 commits into
masterfrom
viglia/feat/react-native-mixed-frames
Nov 7, 2023
Merged

ref(profiling): support symbolication of native frames in react-native#58771
viglia merged 14 commits into
masterfrom
viglia/feat/react-native-mixed-frames

Conversation

@viglia

@viglia viglia commented Oct 25, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@viglia viglia self-assigned this Oct 25, 2023
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 25, 2023
@codecov

codecov Bot commented Oct 25, 2023

Copy link
Copy Markdown

Codecov Report

Merging #58771 (a68219d) into master (6e656e1) will increase coverage by 5.21%.
The diff coverage is 16.66%.

❗ Current head a68219d differs from pull request most recent head dd1c2a2. Consider uploading reports for the commit dd1c2a2 to get more accurate results

@@            Coverage Diff             @@
##           master   #58771      +/-   ##
==========================================
+ Coverage   75.52%   80.74%   +5.21%     
==========================================
  Files        5492     5164     -328     
  Lines      235519   226099    -9420     
  Branches    38102    38058      -44     
==========================================
+ Hits       177881   182564    +4683     
+ Misses      56401    38003   -18398     
- Partials     1237     5532    +4295     
Files Coverage Δ
src/sentry/profiles/task.py 40.61% <16.66%> (-8.53%) ⬇️

... and 2021 files with indirect coverage changes

Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py
Comment thread src/sentry/profiles/task.py
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py Outdated
Comment thread src/sentry/profiles/task.py
from symbolicator

Uses _merge_image instead of blindly replacing them with the response
from symbolicator.

In case we have other images than native and sourcemaps, this will also
preserve them.
@krystofwoldrich

Copy link
Copy Markdown
Contributor

Is this PR blocked by something or can we merge it?

@viglia
viglia merged commit d201140 into master Nov 7, 2023
@viglia
viglia deleted the viglia/feat/react-native-mixed-frames branch November 7, 2023 15:51
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants