Skip to content

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

Merged
viglia merged 14 commits intomasterfrom
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 intomasterfrom
viglia/feat/react-native-mixed-frames

Conversation

@viglia
Copy link
Contributor

@viglia viglia commented Oct 25, 2023

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
Copy link

codecov bot commented Oct 25, 2023

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

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
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