-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
Add support for Source Maps Debug ID in Hermes bundle #2947
Closed
5 tasks done
Labels
Comments
github-project-automation
bot
moved this to Needs Discussion
in Mobile & Cross Platform SDK
Mar 31, 2023
krystofwoldrich
moved this from Needs Discussion
to Backlog
in Mobile & Cross Platform SDK
May 12, 2023
@krystofwoldrich you can link it to getsentry/team-mobile#133 |
krystofwoldrich
changed the title
Add support for Source Maps DebugID in Hermes bundle
Add support for Source Maps Debug ID in Hermes bundle
Jun 20, 2023
We might be able to use https://facebook.github.io/metro/docs/configuration#getpolyfills |
This was referenced Jul 5, 2023
github-project-automation
bot
moved this from Needs Review
to Done
in Mobile & Cross Platform SDK
Oct 6, 2023
Hey @krystofwoldrich - I know we have docs on uploading source maps with Hermes, are there any updates needed based on these improvements? |
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Based on the initial testing, the current way of injecting Debug ID into a bundle file works only for plain text JS files.
Tasks
Debug ID must be propagated to Hermes Combined Source Map, but CLI at the moment doesn't automatically upload combined source maps.
Related tasks
The text was updated successfully, but these errors were encountered: