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

feat(js): Read debug IDs from debugId field #874

Merged
merged 4 commits into from
Nov 7, 2024
Merged

Conversation

loewenheim
Copy link
Contributor

@loewenheim loewenheim commented Nov 6, 2024

Companion PR to getsentry/rust-sourcemap#97. This adjusts discover_sourcemap_embedded_debug_id so that it finds debug IDs at both "debug_id" and "debugId".

#skip-changelog because Danger is acting up.

@loewenheim loewenheim requested review from lforst and a team November 6, 2024 15:35
@loewenheim loewenheim self-assigned this Nov 6, 2024
Copy link

github-actions bot commented Nov 6, 2024

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 74170dc

Companion PR to getsentry/rust-sourcemap#97.
This adjusts `discover_sourcemap_embedded_debug_id` so that
it finds debug IDs at both `"debug_id"` and `"debugId"`.
@loewenheim loewenheim enabled auto-merge (squash) November 7, 2024 10:19
@loewenheim loewenheim merged commit b3d4ff9 into master Nov 7, 2024
13 of 14 checks passed
@loewenheim loewenheim deleted the feat/debugId branch November 7, 2024 10:22
loewenheim added a commit to getsentry/sentry-cli that referenced this pull request Nov 7, 2024
This is accomplished by updating `symbolic` and `rust-sourcemap`. See
getsentry/symbolic#874 and
getsentry/rust-sourcemap#97.

Closes #2220.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants