Skip to content

Commit

Permalink
chore(deps): update JavaScript SDK to v7.60.1 (#3199)
Browse files Browse the repository at this point in the history
Co-authored-by: GitHub <[email protected]>
Co-authored-by: Krystof Woldrich <[email protected]>
  • Loading branch information
3 people authored Jul 27, 2023
1 parent 0c6c447 commit 5ce6eed
Show file tree
Hide file tree
Showing 5 changed files with 159 additions and 96 deletions.
6 changes: 3 additions & 3 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,9 @@

### Dependencies

- Bump JavaScript SDK from v7.57.0 to v7.58.1 ([#3184](https://github.com/getsentry/sentry-react-native/pull/3184))
- [changelog](https://github.com/getsentry/sentry-javascript/blob/develop/CHANGELOG.md#7581)
- [diff](https://github.com/getsentry/sentry-javascript/compare/7.57.0...7.58.1)
- Bump JavaScript SDK from v7.57.0 to v7.60.1 ([#3184](https://github.com/getsentry/sentry-react-native/pull/3184), [#3199](https://github.com/getsentry/sentry-react-native/pull/3199))
- [changelog](https://github.com/getsentry/sentry-javascript/blob/develop/CHANGELOG.md#7601)
- [diff](https://github.com/getsentry/sentry-javascript/compare/7.57.0...7.60.1)
- Bump Cocoa SDK from v8.8.0 to v8.9.3 ([#3188](https://github.com/getsentry/sentry-react-native/pull/3188), [#3206](https://github.com/getsentry/sentry-react-native/pull/3206))
- [changelog](https://github.com/getsentry/sentry-cocoa/blob/main/CHANGELOG.md#893)
- [diff](https://github.com/getsentry/sentry-cocoa/compare/8.8.0...8.9.3)
Expand Down
18 changes: 9 additions & 9 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,18 +56,18 @@
"react-native": ">=0.65.0"
},
"dependencies": {
"@sentry/browser": "7.58.1",
"@sentry/browser": "7.60.1",
"@sentry/cli": "2.19.4",
"@sentry/core": "7.58.1",
"@sentry/hub": "7.58.1",
"@sentry/integrations": "7.58.1",
"@sentry/react": "7.58.1",
"@sentry/types": "7.58.1",
"@sentry/utils": "7.58.1"
"@sentry/core": "7.60.1",
"@sentry/hub": "7.60.1",
"@sentry/integrations": "7.60.1",
"@sentry/react": "7.60.1",
"@sentry/types": "7.60.1",
"@sentry/utils": "7.60.1"
},
"devDependencies": {
"@sentry-internal/eslint-config-sdk": "7.58.1",
"@sentry-internal/eslint-plugin-sdk": "7.58.1",
"@sentry-internal/eslint-config-sdk": "7.60.1",
"@sentry-internal/eslint-plugin-sdk": "7.60.1",
"@sentry/typescript": "^5.20.1",
"@sentry/wizard": "3.2.3",
"@types/jest": "^29.2.5",
Expand Down
4 changes: 2 additions & 2 deletions src/js/profiling/convertHermesProfile.ts
Original file line number Diff line number Diff line change
Expand Up @@ -139,8 +139,8 @@ function mapFrames(hermesStackFrames: Record<Hermes.StackFrameId, Hermes.StackFr
frames.push({
function: functionName || ANONYMOUS_FUNCTION_NAME,
file: hermesFrame.category == 'JavaScript' ? DEFAULT_BUNDLE_NAME : undefined,
line: hermesFrame.line !== undefined ? Number(hermesFrame.line) : undefined,
column: hermesFrame.column !== undefined ? Number(hermesFrame.column) : undefined,
lineno: hermesFrame.line !== undefined ? Number(hermesFrame.line) : undefined,
colno: hermesFrame.column !== undefined ? Number(hermesFrame.column) : undefined,
});
}

Expand Down
16 changes: 8 additions & 8 deletions test/profiling/convertHermesProfile.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -82,28 +82,28 @@ describe('convert hermes profile to sentry profile', () => {
const expectedSentryProfile: ThreadCpuProfile = {
frames: [
{
column: undefined,
colno: undefined,
file: undefined,
function: '[root]',
line: undefined,
lineno: undefined,
},
{
column: 33,
colno: 33,
file: 'app:///main.jsbundle',
function: 'fooA',
line: 1610,
lineno: 1610,
},
{
column: 21,
colno: 21,
file: 'app:///main.jsbundle',
function: 'fooB',
line: 1616,
lineno: 1616,
},
{
column: 18,
colno: 18,
file: 'app:///main.jsbundle',
function: 'anonymous',
line: 1627,
lineno: 1627,
},
],
samples: [
Expand Down
Loading

0 comments on commit 5ce6eed

Please sign in to comment.