Skip to content
New issue

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

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

Merged
merged 14 commits into from
Nov 7, 2023

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

src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Show resolved Hide resolved
src/sentry/profiles/task.py Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Outdated Show resolved Hide resolved
src/sentry/profiles/task.py Show resolved Hide resolved
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
Member

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